diff options
author | Bananeweizen <bananeweizen@gmx.de> | 2011-11-19 20:09:30 +0100 |
---|---|---|
committer | Bananeweizen <bananeweizen@gmx.de> | 2011-11-19 20:09:30 +0100 |
commit | 13cebd799c30d0542c44fdb4cdffbbef37cfa86c (patch) | |
tree | 2fb4a238a2ee370c150eeead8e28780b696d8325 /main/src/cgeo/geocaching/maps/MapProviderFactory.java | |
parent | c1c685765acb471b3399046e3fc4c4cd415e015d (diff) | |
download | cgeo-13cebd799c30d0542c44fdb4cdffbbef37cfa86c.zip cgeo-13cebd799c30d0542c44fdb4cdffbbef37cfa86c.tar.gz cgeo-13cebd799c30d0542c44fdb4cdffbbef37cfa86c.tar.bz2 |
fix several FindBugs warnings
* methods starting with uppercase letter
* unused field and access methods
* null value not explicitly given as null argument in method call
* unnecessary math.floor()
Diffstat (limited to 'main/src/cgeo/geocaching/maps/MapProviderFactory.java')
-rw-r--r-- | main/src/cgeo/geocaching/maps/MapProviderFactory.java | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/main/src/cgeo/geocaching/maps/MapProviderFactory.java b/main/src/cgeo/geocaching/maps/MapProviderFactory.java index 95be0f3..c7865f3 100644 --- a/main/src/cgeo/geocaching/maps/MapProviderFactory.java +++ b/main/src/cgeo/geocaching/maps/MapProviderFactory.java @@ -44,13 +44,13 @@ public class MapProviderFactory { return getInstance().mapSources; } - public static boolean IsValidSourceId(int sourceId) { + public static boolean isValidSourceId(int sourceId) { return getInstance().mapSources.containsKey(sourceId); } - public static boolean IsSameProvider(int sourceId1, int sourceId2) { + public static boolean isSameProvider(int sourceId1, int sourceId2) { for (MapProvider mp : getInstance().mapProviders) { - if (mp.IsMySource(sourceId1) && mp.IsMySource(sourceId2)) { + if (mp.isMySource(sourceId1) && mp.isMySource(sourceId2)) { return true; } } @@ -59,7 +59,7 @@ public class MapProviderFactory { public static MapProvider getMapProvider(int sourceId) { for (MapProvider mp : getInstance().mapProviders) { - if (mp.IsMySource(sourceId)) { + if (mp.isMySource(sourceId)) { return mp; } } @@ -67,7 +67,6 @@ public class MapProviderFactory { } public static int getSourceOrdinalFromId(int sourceId) { - int sourceOrdinal = 0; for (int key : getInstance().mapSources.keySet()) { if (sourceId == key) { @@ -90,7 +89,6 @@ public class MapProviderFactory { } public static void addMapviewMenuItems(Menu parentMenu, int groupId, int currentSource) { - SortedMap<Integer, String> mapSources = getInstance().mapSources; for (int key : mapSources.keySet()) { |