aboutsummaryrefslogtreecommitdiffstats
path: root/main/src/cgeo/geocaching/maps/MapProviderFactory.java
diff options
context:
space:
mode:
authorBananeweizen <bananeweizen@gmx.de>2011-11-19 20:09:30 +0100
committerBananeweizen <bananeweizen@gmx.de>2011-11-19 20:09:30 +0100
commit13cebd799c30d0542c44fdb4cdffbbef37cfa86c (patch)
tree2fb4a238a2ee370c150eeead8e28780b696d8325 /main/src/cgeo/geocaching/maps/MapProviderFactory.java
parentc1c685765acb471b3399046e3fc4c4cd415e015d (diff)
downloadcgeo-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.java10
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()) {