diff options
author | Bananeweizen <Bananeweizen@gmx.de> | 2014-01-01 14:08:11 +0100 |
---|---|---|
committer | Bananeweizen <Bananeweizen@gmx.de> | 2014-01-01 14:08:11 +0100 |
commit | 9bed9f046731e71f4ef79502ceec46f9502b4de2 (patch) | |
tree | 720a3d7653596bc48fb817cbc8bcbef7029af51c /main/src/cgeo/geocaching/AboutActivity.java | |
parent | 08fde3648e575a7eead15bd830b4936451275e14 (diff) | |
download | cgeo-9bed9f046731e71f4ef79502ceec46f9502b4de2.zip cgeo-9bed9f046731e71f4ef79502ceec46f9502b4de2.tar.gz cgeo-9bed9f046731e71f4ef79502ceec46f9502b4de2.tar.bz2 |
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.
Diffstat (limited to 'main/src/cgeo/geocaching/AboutActivity.java')
-rw-r--r-- | main/src/cgeo/geocaching/AboutActivity.java | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/main/src/cgeo/geocaching/AboutActivity.java b/main/src/cgeo/geocaching/AboutActivity.java index b9f198a..b47091b 100644 --- a/main/src/cgeo/geocaching/AboutActivity.java +++ b/main/src/cgeo/geocaching/AboutActivity.java @@ -179,9 +179,8 @@ public class AboutActivity extends AbstractViewPagerActivity<AboutActivity.Page> return new ContributorsViewCreator(); case LICENSE: return new LicenseViewCreator(); - default: - throw new IllegalArgumentException(); } + throw new IllegalStateException(); // cannot happen, when switch case is enum complete } @Override |