diff options
| author | Bananeweizen <bananeweizen@gmx.de> | 2013-08-18 18:44:50 +0200 |
|---|---|---|
| committer | Bananeweizen <bananeweizen@gmx.de> | 2013-08-18 18:44:50 +0200 |
| commit | e3bad494f541eb574e80e466ed8abaef8bb5d257 (patch) | |
| tree | f3efbacc2d7c7b8940c5410daab85cd8b4852518 /main/src/cgeo/geocaching/network/HtmlImage.java | |
| parent | 8fd7c4eedabbf77c9d4131503d122c55a23b070f (diff) | |
| download | cgeo-e3bad494f541eb574e80e466ed8abaef8bb5d257.zip cgeo-e3bad494f541eb574e80e466ed8abaef8bb5d257.tar.gz cgeo-e3bad494f541eb574e80e466ed8abaef8bb5d257.tar.bz2 | |
refactoring: fix findbugs issues
Diffstat (limited to 'main/src/cgeo/geocaching/network/HtmlImage.java')
| -rw-r--r-- | main/src/cgeo/geocaching/network/HtmlImage.java | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/main/src/cgeo/geocaching/network/HtmlImage.java b/main/src/cgeo/geocaching/network/HtmlImage.java index 0649e12..797e67d 100644 --- a/main/src/cgeo/geocaching/network/HtmlImage.java +++ b/main/src/cgeo/geocaching/network/HtmlImage.java @@ -6,6 +6,7 @@ import cgeo.geocaching.cgeoapplication; import cgeo.geocaching.compatibility.Compatibility; import cgeo.geocaching.connector.ConnectorFactory; import cgeo.geocaching.files.LocalStorage; +import cgeo.geocaching.utils.FileUtils; import cgeo.geocaching.utils.IOUtils; import cgeo.geocaching.utils.ImageUtils; import cgeo.geocaching.utils.Log; @@ -146,9 +147,13 @@ public class HtmlImage implements Html.ImageGetter { */ private static void makeFreshCopy(final File file) { final File tempFile = new File(file.getParentFile(), file.getName() + "-temp"); - file.renameTo(tempFile); - LocalStorage.copy(tempFile, file); - tempFile.delete(); + if (file.renameTo(tempFile)) { + LocalStorage.copy(tempFile, file); + FileUtils.deleteIgnoringFailure(tempFile); + } + else { + Log.e("Could not reset timestamp of file " + file.getAbsolutePath()); + } } private Bitmap getTransparent1x1Image() { |
