From cd03c1b91da2b5c8cc7eb4f5617263c6aebcbd21 Mon Sep 17 00:00:00 2001
From: Bananeweizen <bananeweizen@gmx.de>
Date: Sun, 18 Aug 2013 19:08:39 +0200
Subject: refactoring: more findbugs cleanup

---
 main/src/cgeo/geocaching/files/LocalStorage.java | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'main/src/cgeo/geocaching/files/LocalStorage.java')

diff --git a/main/src/cgeo/geocaching/files/LocalStorage.java b/main/src/cgeo/geocaching/files/LocalStorage.java
index 7657e1e..57f586e 100644
--- a/main/src/cgeo/geocaching/files/LocalStorage.java
+++ b/main/src/cgeo/geocaching/files/LocalStorage.java
@@ -167,7 +167,7 @@ public final class LocalStorage {
 
     private static File buildFile(final File base, final String fileName, final boolean isUrl, final boolean createDirs) {
         if (createDirs) {
-            base.mkdirs();
+            FileUtils.mkdirs(base);
         }
         return new File(base, isUrl ? CryptUtils.md5(fileName) + getExtension(fileName) : fileName);
     }
@@ -287,7 +287,7 @@ public final class LocalStorage {
      * @return true if the copy happened without error, false otherwise
      */
     public static boolean copy(final File source, final File destination) {
-        destination.getParentFile().mkdirs();
+        FileUtils.mkdirs(destination.getParentFile());
 
         InputStream input = null;
         OutputStream output = null;
@@ -296,9 +296,10 @@ public final class LocalStorage {
             output = new BufferedOutputStream(new FileOutputStream(destination));
         } catch (FileNotFoundException e) {
             Log.e("LocalStorage.copy: could not open file", e);
+            return false;
+        } finally {
             IOUtils.closeQuietly(input);
             IOUtils.closeQuietly(output);
-            return false;
         }
 
         boolean copyDone = copy(input, output);
-- 
cgit v1.1