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/TrackableActivity.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/TrackableActivity.java')
| -rw-r--r-- | main/src/cgeo/geocaching/TrackableActivity.java | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/main/src/cgeo/geocaching/TrackableActivity.java b/main/src/cgeo/geocaching/TrackableActivity.java index 6752344..973f8c2 100644 --- a/main/src/cgeo/geocaching/TrackableActivity.java +++ b/main/src/cgeo/geocaching/TrackableActivity.java @@ -316,9 +316,8 @@ public class TrackableActivity extends AbstractViewPagerActivity<TrackableActivi return new DetailsViewCreator(); case LOGS: return new TrackableLogsViewCreator(this, trackable); - default: - throw new IllegalArgumentException(); } + throw new IllegalStateException(); // cannot happen as long as switch case is enum complete } @Override |
