diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2012-12-19 20:48:05 +0100 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2012-12-19 20:48:05 +0100 |
| commit | b61038cab2ad20899f816229f04223eae7c317e1 (patch) | |
| tree | 42e2065cf2e51314aaeb191856d4bbfe95eb3d69 /main | |
| parent | c0c92e54755d6603785c4de37c70ae78efe880b1 (diff) | |
| download | cgeo-b61038cab2ad20899f816229f04223eae7c317e1.zip cgeo-b61038cab2ad20899f816229f04223eae7c317e1.tar.gz cgeo-b61038cab2ad20899f816229f04223eae7c317e1.tar.bz2 | |
Refactoring: style issues
Add missing spaces around assignment, use a if instead of a two cases
switch since there are only two alternatives.
Diffstat (limited to 'main')
| -rw-r--r-- | main/src/cgeo/geocaching/CacheDetailActivity.java | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/main/src/cgeo/geocaching/CacheDetailActivity.java b/main/src/cgeo/geocaching/CacheDetailActivity.java index 6c33dde..512f11d 100644 --- a/main/src/cgeo/geocaching/CacheDetailActivity.java +++ b/main/src/cgeo/geocaching/CacheDetailActivity.java @@ -2546,20 +2546,16 @@ public class CacheDetailActivity extends AbstractActivity { public void onClick(View v) { dismiss(); final ProgressDialog p = ProgressDialog.show(CacheDetailActivity.this, res.getString(R.string.cache), res.getString(R.string.waypoint_reset), true); - Handler h = new Handler() { - private boolean remoteFinished=false; - private boolean localFinished=false; + final Handler h = new Handler() { + private boolean remoteFinished = false; + private boolean localFinished = false; @Override public void handleMessage(Message msg) { - switch (msg.what) { - case ResetCoordsThread.LOCAL: - localFinished=true; - break; - case ResetCoordsThread.ON_WEBSITE: - remoteFinished=true; - break; - + if (msg.what == ResetCoordsThread.LOCAL) { + localFinished = true; + } else { + remoteFinished = true; } if ((localFinished || !resetLocalyOption.isChecked()) && (remoteFinished || !uploadOption.isChecked())) { |