diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2014-04-15 12:12:02 +0200 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2014-04-15 12:12:02 +0200 |
| commit | 540c45191c819d5498a1075d7f38e0ed6501850e (patch) | |
| tree | b7204c73163ffbf99a8b126d79a339cff0fcf4f6 /main/src | |
| parent | 82d2f14fb418df76bd3bf2433c108a08351e2bfd (diff) | |
| download | cgeo-540c45191c819d5498a1075d7f38e0ed6501850e.zip cgeo-540c45191c819d5498a1075d7f38e0ed6501850e.tar.gz cgeo-540c45191c819d5498a1075d7f38e0ed6501850e.tar.bz2 | |
Help FindBugs another way
Diffstat (limited to 'main/src')
| -rw-r--r-- | main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java index d6ce598..b2ce11a 100644 --- a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java +++ b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java @@ -359,11 +359,11 @@ public class CoordinatesInputDialog extends NoTitleDialog { current = new Geopoint(latDir, latDeg, latMin, latSec, latSecFrac, lonDir, lonDeg, lonMin, lonSec, lonSecFrac); break; case Plain: - // This case has been handled above. Help FindBugs by asserting that current cannot be null even here - assert current != null; + // This case has been handled above } } - if (current.isValid()) { + // The null check is necessary to keep FindBugs happy + if (current != null && current.isValid()) { gp = current; return true; } |
