refactoring: remove bad default case statements

For methods returning directly from the switch case, at least Eclipse
still needs a useless return value outside of the switch statement. This
has been handled by IllegalStateExceptions after the switch.

1 files changed, 1 insertions, 4 deletions

diff --git a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java
index ea7554d..e00993f 100644
--- a/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java
+++ b/main/src/cgeo/geocaching/ui/dialog/CoordinatesInputDialog.java
@@ -216,9 +216,8 @@ public class CoordinatesInputDialog extends NoTitleDialog {
                 eLonSec.setText(addZeros(gp.getLonSec(), 2));
                 eLonSub.setText(addZeros(gp.getLonSecFrac(), 3));
                 break;
-            default:
-                throw new IllegalStateException();
         }
+        throw new IllegalStateException(); // cannot happen if switch case is enum complete
     }
 
     private static String addZeros(final int value, final int len) {
@@ -361,8 +360,6 @@ public class CoordinatesInputDialog extends NoTitleDialog {
                     break;
                 case Plain:
                     // This case has been handled above
-                default:
-                    throw new IllegalArgumentException();
             }
         } catch (final Geopoint.ParseException e) {
             if (signalError) {