refactoring: use exception object in loggging

31 files changed, 71 insertions, 72 deletions

diff --git a/main/src/cgeo/geocaching/CachePopup.java b/main/src/cgeo/geocaching/CachePopup.java
index 9b0d96d..4bda294 100644
--- a/main/src/cgeo/geocaching/CachePopup.java
+++ b/main/src/cgeo/geocaching/CachePopup.java
@@ -132,7 +132,7 @@ public class CachePopup extends AbstractPopupActivity {
                 offlineStore.setOnClickListener(new StoreCacheClickListener());
             }
         } catch (Exception e) {
-            Log.e("cgeopopup.init: " + e.toString());
+            Log.e("cgeopopup.init", e);
         }
 
         // cache is loaded. remove progress-popup if any there
diff --git a/main/src/cgeo/geocaching/EditWaypointActivity.java b/main/src/cgeo/geocaching/EditWaypointActivity.java
index f64e6df..3958d5f 100644
--- a/main/src/cgeo/geocaching/EditWaypointActivity.java
+++ b/main/src/cgeo/geocaching/EditWaypointActivity.java
@@ -91,7 +91,7 @@ public class EditWaypointActivity extends AbstractActivity {
 
                 initializeDistanceUnitSelector();
             } catch (Exception e) {
-                Log.e("EditWaypointActivity.loadWaypointHandler: " + e.toString());
+                Log.e("EditWaypointActivity.loadWaypointHandler", e);
             } finally {
                 if (waitDialog != null) {
                     waitDialog.dismiss();
@@ -275,7 +275,7 @@ public class EditWaypointActivity extends AbstractActivity {
 
                 loadWaypointHandler.sendMessage(Message.obtain());
             } catch (Exception e) {
-                Log.e("cgeowaypoint.loadWaypoint.run: " + e.toString());
+                Log.e("cgeowaypoint.loadWaypoint.run", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/LogTrackableActivity.java b/main/src/cgeo/geocaching/LogTrackableActivity.java
index 168b1ee..0217c08 100644
--- a/main/src/cgeo/geocaching/LogTrackableActivity.java
+++ b/main/src/cgeo/geocaching/LogTrackableActivity.java
@@ -311,7 +311,7 @@ public class LogTrackableActivity extends AbstractLoggingActivity implements Dat
                     possibleLogTypes.addAll(typesPre);
                 }
             } catch (Exception e) {
-                Log.e("LogTrackableActivity.LoadDataThread.run: " + e.toString());
+                Log.e("LogTrackableActivity.LoadDataThread.run", e);
             }
 
             loadDataHandler.sendEmptyMessage(0);
@@ -349,7 +349,7 @@ public class LogTrackableActivity extends AbstractLoggingActivity implements Dat
 
             return status;
         } catch (Exception e) {
-            Log.e("LogTrackableActivity.postLogFn: " + e.toString());
+            Log.e("LogTrackableActivity.postLogFn", e);
         }
 
         return StatusCode.LOG_POST_ERROR;
diff --git a/main/src/cgeo/geocaching/SettingsActivity.java b/main/src/cgeo/geocaching/SettingsActivity.java
index 0a41942..64a086d 100644
--- a/main/src/cgeo/geocaching/SettingsActivity.java
+++ b/main/src/cgeo/geocaching/SettingsActivity.java
@@ -84,7 +84,7 @@ public class SettingsActivity extends AbstractActivity {
             } catch (Exception e) {
                 showToast(res.getString(R.string.err_login_failed));
 
-                Log.e("SettingsActivity.logInHandler: " + e.toString());
+                Log.e("SettingsActivity.logInHandler", e);
             }
 
             if (loginDialog != null && loginDialog.isShowing()) {
@@ -112,7 +112,7 @@ public class SettingsActivity extends AbstractActivity {
             } catch (Exception e) {
                 showToast(res.getString(R.string.init_sendToCgeo_register_fail));
 
-                Log.e("SettingsActivity.webHandler: " + e.toString());
+                Log.e("SettingsActivity.webHandler", e);
             }
 
             if (webDialog != null && webDialog.isShowing()) {
@@ -952,7 +952,7 @@ public class SettingsActivity extends AbstractActivity {
                         try {
                             pin = Integer.parseInt(strings[1].trim());
                         } catch (Exception e) {
-                            Log.e("webDialog: " + e.toString());
+                            Log.e("webDialog", e);
                         }
                         String code = strings[0];
                         Settings.setWebNameCode(nam, code);
diff --git a/main/src/cgeo/geocaching/StaticMapsActivity.java b/main/src/cgeo/geocaching/StaticMapsActivity.java
index a15aa73..263b6b8 100644
--- a/main/src/cgeo/geocaching/StaticMapsActivity.java
+++ b/main/src/cgeo/geocaching/StaticMapsActivity.java
@@ -59,7 +59,7 @@ public class StaticMapsActivity extends AbstractActivity {
                     showStaticMaps();
                 }
             } catch (Exception e) {
-                Log.e("StaticMapsActivity.loadMapsHandler: " + e.toString());
+                Log.e("StaticMapsActivity.loadMapsHandler", e);
             }
         }
     };
@@ -145,7 +145,7 @@ public class StaticMapsActivity extends AbstractActivity {
                                 }
                             }
                         } catch (Exception e) {
-                            Log.e("StaticMapsActivity.LoadMapsThread.run: " + e.toString());
+                            Log.e("StaticMapsActivity.LoadMapsThread.run", e);
                         }
                     }
                     if (!maps.isEmpty()) {
@@ -155,7 +155,7 @@ public class StaticMapsActivity extends AbstractActivity {
 
                 loadMapsHandler.sendMessage(Message.obtain());
             } catch (Exception e) {
-                Log.e("StaticMapsActivity.LoadMapsThread.run: " + e.toString());
+                Log.e("StaticMapsActivity.LoadMapsThread.run", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/StaticMapsProvider.java b/main/src/cgeo/geocaching/StaticMapsProvider.java
index d587efd..b418400 100644
--- a/main/src/cgeo/geocaching/StaticMapsProvider.java
+++ b/main/src/cgeo/geocaching/StaticMapsProvider.java
@@ -188,7 +188,7 @@ public class StaticMapsProvider {
             try {
                 pool.add(currentTask, 20, TimeUnit.SECONDS);
             } catch (InterruptedException e) {
-                Log.e("StaticMapsProvider.downloadMaps error adding task: " + e.toString());
+                Log.e("StaticMapsProvider.downloadMaps error adding task", e);
             }
         }
     }
@@ -218,7 +218,7 @@ public class StaticMapsProvider {
             try {
                 StaticMapsProvider.getMapFile(geocode, WAYPOINT_PREFIX + wp_id + '_' + level, false).delete();
             } catch (Exception e) {
-                Log.e("StaticMapsProvider.removeWpStaticMaps: " + e.toString());
+                Log.e("StaticMapsProvider.removeWpStaticMaps", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/VisitCacheActivity.java b/main/src/cgeo/geocaching/VisitCacheActivity.java
index e2b74fa..a29256c 100644
--- a/main/src/cgeo/geocaching/VisitCacheActivity.java
+++ b/main/src/cgeo/geocaching/VisitCacheActivity.java
@@ -557,7 +557,7 @@ public class VisitCacheActivity extends AbstractLoggingActivity implements DateD
 
             return status;
         } catch (Exception e) {
-            Log.e("cgeovisit.postLogFn: " + e.toString());
+            Log.e("cgeovisit.postLogFn", e);
         }
 
         return StatusCode.LOG_POST_ERROR;
diff --git a/main/src/cgeo/geocaching/WaypointPopup.java b/main/src/cgeo/geocaching/WaypointPopup.java
index cc99abd..a69b729 100644
--- a/main/src/cgeo/geocaching/WaypointPopup.java
+++ b/main/src/cgeo/geocaching/WaypointPopup.java
@@ -73,7 +73,7 @@ public class WaypointPopup extends AbstractPopupActivity {
             addCacheDetails();
 
         } catch (Exception e) {
-            Log.e("cgeopopup.init: " + e.toString());
+            Log.e("cgeopopup.init", e);
         }
     }
 
diff --git a/main/src/cgeo/geocaching/apps/cachelist/CacheListAppFactory.java b/main/src/cgeo/geocaching/apps/cachelist/CacheListAppFactory.java
index fbfb9fc..b01c0ac 100644
--- a/main/src/cgeo/geocaching/apps/cachelist/CacheListAppFactory.java
+++ b/main/src/cgeo/geocaching/apps/cachelist/CacheListAppFactory.java
@@ -64,7 +64,7 @@ public final class CacheListAppFactory extends AbstractAppFactory {
                 activity.invalidateOptionsMenuCompatible();
                 return result;
             } catch (Exception e) {
-                Log.e("CacheListAppFactory.onMenuItemSelected: " + e.toString());
+                Log.e("CacheListAppFactory.onMenuItemSelected", e);
             }
         }
         return false;
diff --git a/main/src/cgeo/geocaching/cgCache.java b/main/src/cgeo/geocaching/cgCache.java
index a2a4422..24bd66b 100644
--- a/main/src/cgeo/geocaching/cgCache.java
+++ b/main/src/cgeo/geocaching/cgCache.java
@@ -1333,7 +1333,7 @@ public class cgCache implements ICache, IWaypoint {
                 matcher = new MatcherWrapper(coordPattern, note);
             }
         } catch (Exception e) {
-            Log.e("cgCache.parseWaypointsFromNote: " + e.toString());
+            Log.e("cgCache.parseWaypointsFromNote", e);
         }
     }
 
diff --git a/main/src/cgeo/geocaching/cgeo.java b/main/src/cgeo/geocaching/cgeo.java
index 29c287a..0091d7c 100644
--- a/main/src/cgeo/geocaching/cgeo.java
+++ b/main/src/cgeo/geocaching/cgeo.java
@@ -164,7 +164,7 @@ public class cgeo extends AbstractActivity {
                     showToast(res.getString(reason == StatusCode.MAINTENANCE ? reason.getErrorString() : R.string.err_login_failed_toast));
                 }
             } catch (Exception e) {
-                Log.w("cgeo.firstLoginHander: " + e.toString());
+                Log.w("cgeo.firstLoginHander", e);
             }
         }
     };
@@ -663,7 +663,7 @@ public class cgeo extends AbstractActivity {
                         countBubble.setVisibility(View.VISIBLE);
                     }
                 } catch (Exception e) {
-                    Log.w("cgeo.countBubbleHander: " + e.toString());
+                    Log.w("cgeo.countBubbleHander", e);
                 }
             }
         };
diff --git a/main/src/cgeo/geocaching/cgeocaches.java b/main/src/cgeo/geocaching/cgeocaches.java
index 54b863c..fa83a60 100644
--- a/main/src/cgeo/geocaching/cgeocaches.java
+++ b/main/src/cgeo/geocaching/cgeocaches.java
@@ -206,7 +206,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
             setAdapterCurrentCoordinates(false);
         } catch (Exception e) {
             showToast(res.getString(R.string.err_detail_cache_find_any));
-            Log.e("cgeocaches.loadCachesHandler: " + e.toString());
+            Log.e("cgeocaches.loadCachesHandler", e);
 
             hideLoading();
             showProgress(false);
@@ -219,7 +219,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
             hideLoading();
             showProgress(false);
         } catch (Exception e2) {
-            Log.e("cgeocaches.loadCachesHandler.2: " + e2.toString());
+            Log.e("cgeocaches.loadCachesHandler.2", e2);
         }
 
         adapter.setSelectMode(false);
@@ -269,7 +269,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
                 setAdapterCurrentCoordinates(false);
             } catch (Exception e) {
                 showToast(res.getString(R.string.err_detail_cache_find_next));
-                Log.e("cgeocaches.loadNextPageHandler: " + e.toString());
+                Log.e("cgeocaches.loadNextPageHandler", e);
             }
 
             hideLoading();
@@ -947,7 +947,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
         try {
             adapterInfo = (AdapterContextMenuInfo) info;
         } catch (Exception e) {
-            Log.w("cgeocaches.onCreateContextMenu: " + e.toString());
+            Log.w("cgeocaches.onCreateContextMenu", e);
         }
 
         if (adapterInfo == null || adapterInfo.position >= adapter.getCount()) {
@@ -1003,7 +1003,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
         try {
             adapterInfo = (AdapterContextMenuInfo) info;
         } catch (Exception e) {
-            Log.w("cgeocaches.onContextItemSelected: " + e.toString());
+            Log.w("cgeocaches.onContextItemSelected", e);
         }
 
         final cgCache cache = adapterInfo != null ? getCacheFromAdapter(adapterInfo) : null;
@@ -1483,7 +1483,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
 
                         Log.i("Waiting for next cache " + delay + " ms");
                     } catch (Exception e) {
-                        Log.e("cgeocaches.LoadDetailsThread.sleep: " + e.toString());
+                        Log.e("cgeocaches.LoadDetailsThread.sleep", e);
                     }
                 }
 
@@ -1501,7 +1501,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
                 Log.i(e.getMessage());
                 return false;
             } catch (Exception e) {
-                Log.e("cgeocaches.LoadDetailsThread: " + e.toString());
+                Log.e("cgeocaches.LoadDetailsThread", e);
             }
 
             last = System.currentTimeMillis();
@@ -1583,7 +1583,7 @@ public class cgeocaches extends AbstractListActivity implements FilteredActivity
                         times = 0;
                     }
                 } catch (InterruptedException e) {
-                    Log.e("cgeocaches.LoadFromWebThread.sleep: " + e.toString());
+                    Log.e("cgeocaches.LoadFromWebThread.sleep", e);
                 }
             }
 
diff --git a/main/src/cgeo/geocaching/connector/gc/GCParser.java b/main/src/cgeo/geocaching/connector/gc/GCParser.java
index 108698c..274ba0d 100644
--- a/main/src/cgeo/geocaching/connector/gc/GCParser.java
+++ b/main/src/cgeo/geocaching/connector/gc/GCParser.java
@@ -284,7 +284,7 @@ public abstract class GCParser {
                 LocParser.parseLoc(searchResult, coordinates);
 
             } catch (Exception e) {
-                Log.e("GCParser.parseSearch.CIDs: " + e.toString());
+                Log.e("GCParser.parseSearch.CIDs", e);
             }
         }
 
@@ -466,7 +466,7 @@ public abstract class GCParser {
                 cache.setCoords(new Geopoint(cache.getLatlon()));
                 cache.setReliableLatLon(true);
             } catch (Geopoint.GeopointException e) {
-                Log.w("GCParser.parseCache: Failed to parse cache coordinates: " + e.toString());
+                Log.w("GCParser.parseCache: Failed to parse cache coordinates", e);
             }
         }
 
@@ -1036,7 +1036,7 @@ public abstract class GCParser {
                 page = Network.getResponseData(Network.postRequest(uri, params));
             }
         } catch (Exception e) {
-            Log.e("GCParser.postLog.confim: " + e.toString());
+            Log.e("GCParser.postLog.confim", e);
         }
 
         try {
@@ -1057,7 +1057,7 @@ public abstract class GCParser {
                 return StatusCode.NO_ERROR;
             }
         } catch (Exception e) {
-            Log.e("GCParser.postLog.check: " + e.toString());
+            Log.e("GCParser.postLog.check", e);
         }
 
         Log.e("GCParser.postLog: Failed to post log because of unknown error");
@@ -1116,7 +1116,7 @@ public abstract class GCParser {
                 return StatusCode.NO_ERROR;
             }
         } catch (Exception e) {
-            Log.e("GCParser.postLogTrackable.check: " + e.toString());
+            Log.e("GCParser.postLogTrackable.check", e);
         }
 
         Log.e("GCParser.postLogTrackable: Failed to post log because of unknown error");
@@ -1409,7 +1409,7 @@ public abstract class GCParser {
             }
         } catch (Exception e) {
             // failed to parse logs
-            Log.w("GCParser.parseCache: Failed to parse cache logs" + e.toString());
+            Log.w("GCParser.parseCache: Failed to parse cache logs", e);
         }
 
         // tracking code
diff --git a/main/src/cgeo/geocaching/connector/oc/OC11XMLParser.java b/main/src/cgeo/geocaching/connector/oc/OC11XMLParser.java
index 0b91b25..068ac88 100644
--- a/main/src/cgeo/geocaching/connector/oc/OC11XMLParser.java
+++ b/main/src/cgeo/geocaching/connector/oc/OC11XMLParser.java
@@ -41,6 +41,7 @@ public class OC11XMLParser {
     private static final String PARAGRAPH_BEGIN = "<p>";
     private static Pattern STRIP_DATE = Pattern.compile("\\+0([0-9]){1}\\:00");
     private static Pattern LOCAL_URL = Pattern.compile("href=\"(.*)\"");
+    private static final int CACHE_PARSE_LIMIT = 250;
 
     private static class CacheHolder {
         public cgCache cache;
@@ -160,7 +161,6 @@ public class OC11XMLParser {
     }
 
     private static void setCacheStatus(final int statusId, final cgCache cache) {
-
         switch (statusId) {
             case 1:
                 cache.setArchived(false);
@@ -199,8 +199,6 @@ public class OC11XMLParser {
 
     public static Collection<cgCache> parseCaches(final InputStream stream) throws IOException {
 
-        final int CACHE_PARSE_LIMIT = 250;
-
         final Map<String, cgCache> caches = new HashMap<String, cgCache>();
 
         final CacheHolder cacheHolder = new CacheHolder();
@@ -536,7 +534,7 @@ public class OC11XMLParser {
             Xml.parse(stream, Xml.Encoding.UTF_8, root.getContentHandler());
             return caches.values();
         } catch (SAXException e) {
-            Log.e("Cannot parse .gpx file as oc11xml: could not parse XML - " + e.toString());
+            Log.e("Cannot parse .gpx file as oc11xml: could not parse XML", e);
             return null;
         }
     }
@@ -557,7 +555,8 @@ public class OC11XMLParser {
     }
 
     /**
-     * removes unneeded markup
+     * Removes unneeded markup. Log texts are typically encapsulated in paragraph tags which lead to more empty space on
+     * rendering.
      */
     protected static String stripMarkup(String input) {
         if (StringUtils.startsWith(input, PARAGRAPH_BEGIN) && StringUtils.endsWith(input, PARAGRAPH_END)) {
diff --git a/main/src/cgeo/geocaching/files/FileList.java b/main/src/cgeo/geocaching/files/FileList.java
index 2446834..42c9f16 100644
--- a/main/src/cgeo/geocaching/files/FileList.java
+++ b/main/src/cgeo/geocaching/files/FileList.java
@@ -183,7 +183,7 @@ public abstract class FileList<T extends ArrayAdapter<File>> extends AbstractLis
                     Log.w("No external media mounted.");
                 }
             } catch (Exception e) {
-                Log.e("cgFileList.loadFiles.run: " + e.toString());
+                Log.e("cgFileList.loadFiles.run", e);
             }
 
             changeWaitDialogHandler.sendMessage(Message.obtain(changeWaitDialogHandler, 0, "loaded directories"));
diff --git a/main/src/cgeo/geocaching/files/GPXParser.java b/main/src/cgeo/geocaching/files/GPXParser.java
index 8207455..22a7f41 100644
--- a/main/src/cgeo/geocaching/files/GPXParser.java
+++ b/main/src/cgeo/geocaching/files/GPXParser.java
@@ -362,7 +362,7 @@ public abstract class GPXParser extends FileParser {
                 try {
                     cache.setHidden(parseDate(body));
                 } catch (Exception e) {
-                    Log.w("Failed to parse cache date: " + e.toString());
+                    Log.w("Failed to parse cache date", e);
                 }
             }
         });
@@ -578,7 +578,7 @@ public abstract class GPXParser extends FileParser {
                     try {
                         cache.setDifficulty(Float.parseFloat(body));
                     } catch (NumberFormatException e) {
-                        Log.w("Failed to parse difficulty: " + e.toString());
+                        Log.w("Failed to parse difficulty", e);
                     }
                 }
             });
@@ -591,7 +591,7 @@ public abstract class GPXParser extends FileParser {
                     try {
                         cache.setTerrain(Float.parseFloat(body));
                     } catch (NumberFormatException e) {
-                        Log.w("Failed to parse terrain: " + e.toString());
+                        Log.w("Failed to parse terrain", e);
                     }
                 }
             });
@@ -735,7 +735,7 @@ public abstract class GPXParser extends FileParser {
                     try {
                         log.date = parseDate(body).getTime();
                     } catch (Exception e) {
-                        Log.w("Failed to parse log date: " + e.toString());
+                        Log.w("Failed to parse log date", e);
                     }
                 }
             });
@@ -774,7 +774,7 @@ public abstract class GPXParser extends FileParser {
             Xml.parse(progressStream, Xml.Encoding.UTF_8, root.getContentHandler());
             return cgData.loadCaches(result, EnumSet.of(LoadFlag.LOAD_DB_MINIMAL));
         } catch (SAXException e) {
-            Log.w("Cannot parse .gpx file as GPX " + version + ": could not parse XML - " + e.toString());
+            Log.w("Cannot parse .gpx file as GPX " + version + ": could not parse XML - ", e);
             throw new ParserException("Cannot parse .gpx file as GPX " + version + ": could not parse XML", e);
         }
     }
diff --git a/main/src/cgeo/geocaching/files/LocalStorage.java b/main/src/cgeo/geocaching/files/LocalStorage.java
index 3101ae9..f59f15c 100644
--- a/main/src/cgeo/geocaching/files/LocalStorage.java
+++ b/main/src/cgeo/geocaching/files/LocalStorage.java
@@ -363,7 +363,7 @@ public class LocalStorage {
                     Log.w("LocalStorage.deleteFilesPrefix: Can't delete file " + file.getName());
                 }
             } catch (Exception e) {
-                Log.e("LocalStorage.deleteFilesPrefix: " + e.toString());
+                Log.e("LocalStorage.deleteFilesPrefix", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/gcvote/GCVote.java b/main/src/cgeo/geocaching/gcvote/GCVote.java
index e98ba56..fa17775 100644
--- a/main/src/cgeo/geocaching/gcvote/GCVote.java
+++ b/main/src/cgeo/geocaching/gcvote/GCVote.java
@@ -179,7 +179,7 @@ public final class GCVote {
                 }
             }
         } catch (Exception e) {
-            Log.e("GCVote.getRating: " + e.toString());
+            Log.e("GCVote.getRating", e);
         }
 
         return ratings;
@@ -254,7 +254,7 @@ public final class GCVote {
                 }
             }
         } catch (Exception e) {
-            Log.e("GCvote.loadRatings: " + e.toString());
+            Log.e("GCvote.loadRatings", e);
         }
     }
 }
diff --git a/main/src/cgeo/geocaching/geopoint/Geopoint.java b/main/src/cgeo/geocaching/geopoint/Geopoint.java
index 428ebbd..fc5bfbc 100644
--- a/main/src/cgeo/geocaching/geopoint/Geopoint.java
+++ b/main/src/cgeo/geocaching/geopoint/Geopoint.java
@@ -350,7 +350,7 @@ public final class Geopoint implements ICoordinates, Parcelable {
                 return result.getDouble("elevation");
             }
         } catch (Exception e) {
-            Log.w("cgBase.getElevation: " + e.toString());
+            Log.w("cgBase.getElevation", e);
         }
 
         return null;
diff --git a/main/src/cgeo/geocaching/maps/CGeoMap.java b/main/src/cgeo/geocaching/maps/CGeoMap.java
index a470672..47ca240 100644
--- a/main/src/cgeo/geocaching/maps/CGeoMap.java
+++ b/main/src/cgeo/geocaching/maps/CGeoMap.java
@@ -610,7 +610,7 @@ public class CGeoMap extends AbstractMap implements OnMapDragListener, ViewFacto
 
             menu.findItem(SUBMENU_STRATEGY).setEnabled(isLiveEnabled);
         } catch (Exception e) {
-            Log.e("cgeomap.onPrepareOptionsMenu: " + e);
+            Log.e("cgeomap.onPrepareOptionsMenu", e);
         }
 
         return true;
@@ -672,7 +672,7 @@ public class CGeoMap extends AbstractMap implements OnMapDragListener, ViewFacto
 
                                 geoDirUpdate.startDir();
                             } catch (Exception e) {
-                                Log.e("cgeocaches.onPrepareOptionsMenu.onCancel: " + e.toString());
+                                Log.e("cgeocaches.onPrepareOptionsMenu.onCancel", e);
                             }
                         }
                     });
@@ -1086,7 +1086,7 @@ public class CGeoMap extends AbstractMap implements OnMapDragListener, ViewFacto
 
                     yield();
                 } catch (Exception e) {
-                    Log.w("cgeomap.LoadTimer.run: " + e.toString());
+                    Log.w("cgeomap.LoadTimer.run", e);
                 }
             }
         }
@@ -1413,7 +1413,7 @@ public class CGeoMap extends AbstractMap implements OnMapDragListener, ViewFacto
                         cgCache.storeCache(null, geocode, StoredList.STANDARD_LIST_ID, false, handler);
                     }
                 } catch (Exception e) {
-                    Log.e("cgeocaches.LoadDetails.run: " + e.toString());
+                    Log.e("cgeocaches.LoadDetails.run", e);
                 } finally {
                     // one more cache over
                     detailProgress++;
diff --git a/main/src/cgeo/geocaching/maps/CachesOverlay.java b/main/src/cgeo/geocaching/maps/CachesOverlay.java
index 044f69b..fe1a19b 100644
--- a/main/src/cgeo/geocaching/maps/CachesOverlay.java
+++ b/main/src/cgeo/geocaching/maps/CachesOverlay.java
@@ -247,7 +247,7 @@ public class CachesOverlay extends AbstractItemizedOverlay {
 
             progress.dismiss();
         } catch (Exception e) {
-            Log.e("cgMapOverlay.onTap: " + e.toString());
+            Log.e("cgMapOverlay.onTap", e);
             if (progress != null) {
                 progress.dismiss();
             }
@@ -261,7 +261,7 @@ public class CachesOverlay extends AbstractItemizedOverlay {
         try {
             return items.get(index);
         } catch (Exception e) {
-            Log.e("cgMapOverlay.createItem: " + e.toString());
+            Log.e("cgMapOverlay.createItem", e);
         }
 
         return null;
@@ -272,7 +272,7 @@ public class CachesOverlay extends AbstractItemizedOverlay {
         try {
             return items.size();
         } catch (Exception e) {
-            Log.e("cgMapOverlay.size: " + e.toString());
+            Log.e("cgMapOverlay.size", e);
         }
 
         return 0;
diff --git a/main/src/cgeo/geocaching/maps/google/GoogleMapView.java b/main/src/cgeo/geocaching/maps/google/GoogleMapView.java
index 9c134a4..154c3f3 100644
--- a/main/src/cgeo/geocaching/maps/google/GoogleMapView.java
+++ b/main/src/cgeo/geocaching/maps/google/GoogleMapView.java
@@ -61,7 +61,7 @@ public class GoogleMapView extends MapView implements MapViewImpl {
 
             super.draw(canvas);
         } catch (Exception e) {
-            Log.e("GoogleMapView.draw: " + e.toString());
+            Log.e("GoogleMapView.draw", e);
         }
     }
 
@@ -75,7 +75,7 @@ public class GoogleMapView extends MapView implements MapViewImpl {
 
             super.displayZoomControls(takeFocus);
         } catch (Exception e) {
-            Log.e("GoogleMapView.displayZoomControls: " + e.toString());
+            Log.e("GoogleMapView.displayZoomControls", e);
         }
     }
 
diff --git a/main/src/cgeo/geocaching/maps/mapsforge/MapsforgeMapView.java b/main/src/cgeo/geocaching/maps/mapsforge/MapsforgeMapView.java
index 1e4a756..083e5bb 100644
--- a/main/src/cgeo/geocaching/maps/mapsforge/MapsforgeMapView.java
+++ b/main/src/cgeo/geocaching/maps/mapsforge/MapsforgeMapView.java
@@ -59,7 +59,7 @@ public class MapsforgeMapView extends MapView implements MapViewImpl {
 
             super.draw(canvas);
         } catch (Exception e) {
-            Log.e("MapsforgeMapView.draw: " + e.toString());
+            Log.e("MapsforgeMapView.draw", e);
         }
     }
 
@@ -264,7 +264,7 @@ public class MapsforgeMapView extends MapView implements MapViewImpl {
                 }
 
             } catch (Exception e) {
-                Log.e("MapsforgeMapView.repaintRequired: " + e.toString());
+                Log.e("MapsforgeMapView.repaintRequired", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/maps/mapsforge/v024/MapsforgeMapView024.java b/main/src/cgeo/geocaching/maps/mapsforge/v024/MapsforgeMapView024.java
index d177a47..85d61fe 100644
--- a/main/src/cgeo/geocaching/maps/mapsforge/v024/MapsforgeMapView024.java
+++ b/main/src/cgeo/geocaching/maps/mapsforge/v024/MapsforgeMapView024.java
@@ -53,7 +53,7 @@ public class MapsforgeMapView024 extends MapView implements MapViewImpl {
 
             super.draw(canvas);
         } catch (Exception e) {
-            Log.e("MapsforgeMapView.draw: " + e.toString());
+            Log.e("MapsforgeMapView.draw", e);
         }
     }
 
@@ -208,7 +208,7 @@ public class MapsforgeMapView024 extends MapView implements MapViewImpl {
                 }
 
             } catch (Exception e) {
-                Log.e("MapsforgeMapView.repaintRequired: " + e.toString());
+                Log.e("MapsforgeMapView.repaintRequired", e);
             }
         }
     }
diff --git a/main/src/cgeo/geocaching/network/HtmlImage.java b/main/src/cgeo/geocaching/network/HtmlImage.java
index cab92b6..a409750 100644
--- a/main/src/cgeo/geocaching/network/HtmlImage.java
+++ b/main/src/cgeo/geocaching/network/HtmlImage.java
@@ -161,7 +161,7 @@ public class HtmlImage implements Html.ImageGetter {
             final File fileSec = LocalStorage.getStorageSecFile(pseudoGeocode, url, true);
             return loadCachedImage(fileSec, forceKeep);
         } catch (Exception e) {
-            Log.w("HtmlImage.getDrawable (reading cache): " + e.toString());
+            Log.w("HtmlImage.getDrawable (reading cache)", e);
         }
         return null;
     }
diff --git a/main/src/cgeo/geocaching/sorting/AbstractCacheComparator.java b/main/src/cgeo/geocaching/sorting/AbstractCacheComparator.java
index 2015375..aebecd2 100644
--- a/main/src/cgeo/geocaching/sorting/AbstractCacheComparator.java
+++ b/main/src/cgeo/geocaching/sorting/AbstractCacheComparator.java
@@ -19,7 +19,7 @@ public abstract class AbstractCacheComparator implements CacheComparator {
             }
             return compareCaches(cache1, cache2);
         } catch (Exception e) {
-            Log.e("AbstractCacheComparator.compare: " + e.toString());
+            Log.e("AbstractCacheComparator.compare", e);
         }
         return 0;
     }
diff --git a/main/src/cgeo/geocaching/twitter/Twitter.java b/main/src/cgeo/geocaching/twitter/Twitter.java
index 399f852..668502f 100644
--- a/main/src/cgeo/geocaching/twitter/Twitter.java
+++ b/main/src/cgeo/geocaching/twitter/Twitter.java
@@ -43,7 +43,7 @@ public final class Twitter {
                 Log.e("Tweet could not be posted");
             }
         } catch (Exception e) {
-            Log.e("cgBase.postTweet: " + e.toString());
+            Log.e("cgBase.postTweet", e);
         }
     }
 
diff --git a/main/src/cgeo/geocaching/twitter/TwitterAuthorizationActivity.java b/main/src/cgeo/geocaching/twitter/TwitterAuthorizationActivity.java
index 61478b5..3bd6327 100644
--- a/main/src/cgeo/geocaching/twitter/TwitterAuthorizationActivity.java
+++ b/main/src/cgeo/geocaching/twitter/TwitterAuthorizationActivity.java
@@ -162,12 +162,12 @@ public class TwitterAuthorizationActivity extends AbstractActivity {
                         startActivity(new Intent(Intent.ACTION_VIEW, Uri.parse("https://" + host + pathAuthorize + "?" + encodedParams)));
                         status = 1;
                     } catch (Exception e) {
-                        Log.e("TwitterAuthorizationActivity.requestToken(2): " + e.toString());
+                        Log.e("TwitterAuthorizationActivity.requestToken(2)", e);
                     }
                 }
             }
         } catch (Exception e) {
-            Log.e("TwitterAuthorizationActivity.requestToken(1): " + e.toString());
+            Log.e("TwitterAuthorizationActivity.requestToken(1)", e);
         }
 
         requestTokenHandler.sendEmptyMessage(status);
@@ -207,7 +207,7 @@ public class TwitterAuthorizationActivity extends AbstractActivity {
                 status = 1;
             }
         } catch (Exception e) {
-            Log.e("TwitterAuthorizationActivity.changeToken: " + e.toString());
+            Log.e("TwitterAuthorizationActivity.changeToken", e);
         }
 
         changeTokensHandler.sendEmptyMessage(status);
diff --git a/main/src/cgeo/geocaching/ui/CacheListAdapter.java b/main/src/cgeo/geocaching/ui/CacheListAdapter.java
index efc95b8..d6224a9 100644
--- a/main/src/cgeo/geocaching/ui/CacheListAdapter.java
+++ b/main/src/cgeo/geocaching/ui/CacheListAdapter.java
@@ -652,7 +652,7 @@ public class CacheListAdapter extends ArrayAdapter<cgCache> {
                     return true;
                 }
             } catch (Exception e) {
-                Log.w("CacheListAdapter.FlingGesture.onFling: " + e.toString());
+                Log.w("CacheListAdapter.FlingGesture.onFling", e);
             }
 
             return false;
diff --git a/main/src/cgeo/geocaching/ui/ImagesList.java b/main/src/cgeo/geocaching/ui/ImagesList.java
index 00543df..9464114 100644
--- a/main/src/cgeo/geocaching/ui/ImagesList.java
+++ b/main/src/cgeo/geocaching/ui/ImagesList.java
@@ -210,7 +210,7 @@ public class ImagesList {
             fos = new FileOutputStream(file);
             image.getBitmap().compress(CompressFormat.JPEG, 100, fos);
         } catch (Exception e) {
-            Log.e("ImagesActivity.handleMessage.onClick: " + e.toString());
+            Log.e("ImagesActivity.handleMessage.onClick", e);
             return;
         } finally {
             IOUtils.closeQuietly(fos);
diff --git a/main/src/cgeo/geocaching/utils/CryptUtils.java b/main/src/cgeo/geocaching/utils/CryptUtils.java
index 7a23156..df2baa0 100644
--- a/main/src/cgeo/geocaching/utils/CryptUtils.java
+++ b/main/src/cgeo/geocaching/utils/CryptUtils.java
@@ -71,7 +71,7 @@ public final class CryptUtils {
             digest.update(text.getBytes(), 0, text.length());
             hashed = new BigInteger(1, digest.digest()).toString(16);
         } catch (Exception e) {
-            Log.e("cgBase.md5: " + e.toString());
+            Log.e("cgBase.md5", e);
         }
 
         return hashed;
@@ -85,7 +85,7 @@ public final class CryptUtils {
             digest.update(text.getBytes(), 0, text.length());
             hashed = new BigInteger(1, digest.digest()).toString(16);
         } catch (Exception e) {
-            Log.e("cgBase.sha1: " + e.toString());
+            Log.e("cgBase.sha1", e);
         }
 
         return hashed;
@@ -100,7 +100,7 @@ public final class CryptUtils {
             mac.init(secretKeySpec);
             macBytes = mac.doFinal(text.getBytes());
         } catch (Exception e) {
-            Log.e("cgBase.hashHmac: " + e.toString());
+            Log.e("cgBase.hashHmac", e);
         }
 
         return macBytes;