]> hydra-www.ietfng.org Git - acmetensortoys-ctfws-android/commitdiff
treewide: fix inspection-reported issues
authorNathaniel Wesley Filardo <nwfilardo@gmail.com>
Sun, 22 Sep 2019 14:19:34 +0000 (15:19 +0100)
committerNathaniel Wesley Filardo <nwfilardo@gmail.com>
Sun, 22 Sep 2019 14:35:53 +0000 (15:35 +0100)
mobile/src/main/java/com/acmetensortoys/ctfwstimer/AboutActivity.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/AndroidResourceUtils.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/CheckedAsyncDownloader.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/CtFwSCallbacksMQTT.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/CtFwSDisplayLocal.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/CtFwSDisplayTinyChrono.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/HandbookDownloader.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/MainService.java
mobile/src/main/java/com/acmetensortoys/ctfwstimer/MainServiceNotification.java
mobile/src/main/res/layout/activity_main.xml

index 369219aa16d9bc8c4fde381326900340a3dfb327..cb2b39970e615e1300181392aec10026999f4f8e 100644 (file)
@@ -40,7 +40,7 @@ public class AboutActivity extends AppCompatActivity {
     private TextView mTvDebug;
 
     private void makeDebugText() {
-        final StringBuffer sb = new StringBuffer("");
+        final StringBuffer sb = new StringBuffer();
 
         sb.append("Android host version: ");
         sb.append(Build.VERSION.RELEASE);
@@ -93,7 +93,7 @@ public class AboutActivity extends AppCompatActivity {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.activity_about);
 
-        mTvDebug = (TextView) findViewById(R.id.about_debug_tv);
+        mTvDebug = findViewById(R.id.about_debug_tv);
         makeDebugText();
 
         ActionBar ab = getSupportActionBar();
@@ -115,17 +115,17 @@ public class AboutActivity extends AppCompatActivity {
         });
 
         {
-            final WebView wv = (WebView) findViewById(R.id.about_text);
+            final WebView wv = findViewById(R.id.about_text);
             wv.loadData(getResources().getString(R.string.about_text),
                     "text/html", null);
         }
 
         {
-            final WebView wv = (WebView) findViewById(R.id.about_licenses);
+            final WebView wv = findViewById(R.id.about_licenses);
             wv.loadUrl("file:///android_asset/licenses.html");
         }
 
-        TabHost th = (TabHost) findViewById(R.id.about_tab_host);
+        TabHost th = findViewById(R.id.about_tab_host);
         th.setup();
 
         th.addTab(th.newTabSpec(TAB_PROG)
@@ -154,7 +154,7 @@ public class AboutActivity extends AppCompatActivity {
         return true;
     }
 
-    private MainService.Observer mSrvObs = new MainService.Observer() {
+    private final MainService.Observer mSrvObs = new MainService.Observer() {
         @Override
         public void onMqttServerChanged(MainService.LocalBinder b, String sURL) {
 
@@ -172,7 +172,7 @@ public class AboutActivity extends AppCompatActivity {
         }
     };
 
-    private CtFwSGameStateManager.Observer mCtFwSObs = new CtFwSGameStateManager.Observer() {
+    private final CtFwSGameStateManager.Observer mCtFwSObs = new CtFwSGameStateManager.Observer() {
         @Override
         public void onCtFwSConfigure(CtFwSGameStateManager game) {
             makeDebugText();
index ac645718818b5d0d48eac8063c57dd0177e181f6..aee9aa09b73ff5e92b24618021d61fdc603210eb 100644 (file)
@@ -10,6 +10,7 @@ public class AndroidResourceUtils {
         if (Build.VERSION.SDK_INT >= 24) {
             return Html.fromHtml(String.format(rs.getString(id), args), 0);
         } else {
+            //noinspection deprecation
             return Html.fromHtml(String.format(rs.getString(id), args));
         }
     }
index 38c7dab60a1e20a0d47674787c4ee5b7315db8ef..91e415c6b6fee7cdd0dde46fed28aad754f2bf23 100644 (file)
@@ -1,6 +1,5 @@
 package com.acmetensortoys.ctfwstimer;
 
-import android.content.Context;
 import android.os.AsyncTask;
 
 import java.io.BufferedInputStream;
@@ -12,7 +11,6 @@ import java.io.IOException;
 import java.io.OutputStream;
 import java.net.URL;
 import java.net.URLConnection;
-import java.nio.file.NoSuchFileException;
 import java.security.DigestInputStream;
 import java.security.MessageDigest;
 import java.security.NoSuchAlgorithmException;
@@ -75,6 +73,7 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
 
                 byte[] data = new byte[4096];
 
+                //noinspection StatementWithEmptyBody
                 while (is.read(data) != -1) { ; }
 
                 if (java.util.Arrays.equals(is.getMessageDigest().digest(), dl.sha256)) {
@@ -90,6 +89,7 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
                  * Something has gone really wrong.
                  * Unlink the file and try to fetch it
                  */
+                //noinspection ResultOfMethodCallIgnored
                 dl.dest.delete();
             }
 
@@ -109,6 +109,7 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
                 os = new FileOutputStream(oft);
             } catch (IOException ioe) {
                 dl.result = Result.ERR_WRITE;
+                //noinspection ResultOfMethodCallIgnored
                 oft.delete();
                 continue;
             }
@@ -122,6 +123,7 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
                     );
             } catch (IOException ioe) {
                 dl.result = Result.ERR_HOSTUNREACH;
+                //noinspection ResultOfMethodCallIgnored
                 oft.delete();
                 continue;
             }
@@ -138,6 +140,7 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
                         is.close();
                         os.close();
                         dl.result = Result.ERR_TOO_LONG;
+                        //noinspection ResultOfMethodCallIgnored
                         oft.delete();
                         continue dlfor;
                     }
@@ -147,18 +150,21 @@ public class CheckedAsyncDownloader extends AsyncTask<CheckedAsyncDownloader.DL,
                 os.close();
             } catch (IOException ioe) {
                 dl.result = Result.ERR_XFER;
+                //noinspection ResultOfMethodCallIgnored
                 oft.delete();
                 continue;
             }
 
             if (!java.util.Arrays.equals(is.getMessageDigest().digest(), dl.sha256)) {
                 dl.result = Result.ERR_CHECKSUM;
+                //noinspection ResultOfMethodCallIgnored
                 oft.delete();
                 continue;
             }
 
             if (!oft.renameTo(dl.dest)) {
                 dl.result = Result.ERR_WRITE;
+                //noinspection ResultOfMethodCallIgnored
                 oft.delete();
                 continue;
             }
index 69d7e1aec2f83dcef529443cba192d1ebb0551a5..ed53b71327fcbb50174a42b5ce84e23edccf17b4 100644 (file)
@@ -20,7 +20,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onConfig = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             String tm = message.toString().trim();
             Log.d("CtFwS", "Message(Config): " + tm);
             mCgs.fromMqttConfigMessage(tm);
@@ -29,7 +29,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onEnd = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             Log.d("CtFwS", "Message(End): " + message);
             long endT;
             try {
@@ -43,7 +43,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onFlags = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             String tm = message.toString().trim();
             Log.d("CtFwS", "Message(Flags): " + tm);
             mCgs.fromMqttFlagsMessage(tm);
@@ -52,7 +52,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onMessage = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             String str = message.toString();
             Log.d("CtFwS", "Message(Broadcast): " + str);
             mCgs.onNewMessage(str);
@@ -61,7 +61,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onPlayerMessage = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             String str = message.toString();
             Log.d("CtFwS", "Message(Players): " + str);
             mCgs.onNewMessage(str);
@@ -70,7 +70,7 @@ class CtFwSCallbacksMQTT {
 
     final IMqttMessageListener onMessageReset = new IMqttMessageListener() {
         @Override
-        public void messageArrived(String topic, MqttMessage message) throws Exception {
+        public void messageArrived(String topic, MqttMessage message) {
             String str = message.toString();
             Log.d("CtFwS", "Message(Reset): " + str);
             long before;
index 332bcd3087f0c7a4c9101428a0e0beeebfe0ef0d..0b812f97ecab5cb0f1792697313d4c165050b87c 100644 (file)
@@ -6,7 +6,6 @@ import android.graphics.Color;
 import android.os.Build;
 import android.os.Bundle;
 import android.os.SystemClock;
-import android.text.Html;
 import android.text.Spanned;
 import android.text.format.DateUtils;
 import android.util.Log;
@@ -27,7 +26,7 @@ class CtFwSDisplayLocal implements CtFwSGameStateManager.Observer {
     final private Activity mAct;
     String gameStateLabelText;
 
-    private StunTimer stun_short, stun_long;
+    private final StunTimer stun_short, stun_long;
 
     CtFwSDisplayLocal(Activity a) {
         mAct = a;
@@ -103,8 +102,6 @@ class CtFwSDisplayLocal implements CtFwSGameStateManager.Observer {
                         sfx = rs.getString(R.string.notify_game_afoot);
                     }
                     break;
-                case NR_NOT_CONFIG:
-                    // Handled above; fallthru to placate static analysers
                 default:
                     sfx = "";
             }
@@ -394,6 +391,7 @@ class CtFwSDisplayLocal implements CtFwSGameStateManager.Observer {
         final StringBuffer sb = new StringBuffer();
         for (CtFwSGameStateManager.Msg m : msgs) {
 
+            //noinspection StatementWithEmptyBody
             if (m.when == 0 || !gs.isConfigured()) {
                 // leave out the time stamp
             } else if (m.when <= gs.getFirstRoundStartT()) {
index c9627863e332b452869f2de0a476bdc415eac49c..a4802b3dd5e13b013e91ae551d0ca692b4143948 100644 (file)
@@ -6,8 +6,6 @@ import android.widget.Chronometer;
 
 import com.acmetensortoys.ctfwstimer.lib.CtFwSGameStateManager;
 
-import java.util.List;
-import java.util.Locale;
 import java.util.SortedSet;
 
 /*
index c33107f124409e33581aeceb6f616d9a440e7ef1..dae590d5fc4d2c4c4a30fc9891068579d687f256 100644 (file)
@@ -5,11 +5,8 @@ import android.os.Handler;
 import android.support.v4.util.Consumer;
 import android.util.Log;
 
-import com.acmetensortoys.ctfwstimer.CheckedAsyncDownloader;
-
 import org.eclipse.paho.client.mqttv3.IMqttAsyncClient;
 import org.eclipse.paho.client.mqttv3.IMqttMessageListener;
-import org.eclipse.paho.client.mqttv3.MqttClient;
 import org.eclipse.paho.client.mqttv3.MqttException;
 import org.eclipse.paho.client.mqttv3.MqttMessage;
 
@@ -20,14 +17,14 @@ import java.util.Scanner;
 
 public class HandbookDownloader implements IMqttMessageListener {
 
-    private static String TAG = "HandbookDownloader";
+    private static final String TAG = "HandbookDownloader";
 
     private static final long HAND_MAX_LEN = 1024*1024; /* 1 MiB */
 
     private final Context mCtx;
     private final Consumer<CheckedAsyncDownloader.DL> mDLFiniCB;
     private IMqttAsyncClient mMqc;
-    private Handler mHdl;
+    private final Handler mHdl;
     private Runnable nextSubRunnable;
 
     public HandbookDownloader(Context ctx, Handler hdl,
index e0766d1122fbd485e282fd1935c68667165f1a1c..6f0e61450bb1dfae62c7d2f7e38ddfab1d8275bd 100644 (file)
@@ -62,7 +62,7 @@ public class MainService extends Service {
 
     // Handbook fetch logic; this is a singleton for the service, even as connections come and go.
     private CheckedAsyncDownloader.DL lastHandDL;
-    private HandbookDownloader mHandDL = new HandbookDownloader(this,
+    private final HandbookDownloader mHandDL = new HandbookDownloader(this,
             new Handler(Looper.getMainLooper()),
             new Consumer<CheckedAsyncDownloader.DL>() {
                 @Override
index e94894bf1bdf7e212979f3a1cc1167a363cf9288..be9c038d3a70369dfcf915d4ea1a9db3dbfdacde 100644 (file)
@@ -20,7 +20,6 @@ import android.util.Log;
 import com.acmetensortoys.ctfwstimer.lib.CtFwSGameStateManager;
 
 import java.text.NumberFormat;
-import java.util.List;
 import java.util.SortedSet;
 
 class MainServiceNotification {
@@ -105,8 +104,6 @@ class MainServiceNotification {
                     String txt;
                     switch(now.rationale) {
                         default:
-                        case NR_GAME_IN_PROGRESS:
-                        case NR_START_FUTURE:
                             txt = "";
                             break;
                         case NR_TIME_UP:
index 8dcc45dd321aaa4a1963287ad4b1e2a1a973a676..735ee3fe8b2cf0f8b9fb30127756c19d6674aebf 100644 (file)
 
     </LinearLayout>
 
-    <LinearLayout
+    <TableLayout
         android:id="@+id/connmeta"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_alignParentBottom="true"
-        android:orientation="vertical">
+        android:stretchColumns="1">
 
-        <TableLayout
+        <TableRow
             android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            android:stretchColumns="1">
-
-            <TableRow
-                android:layout_width="match_parent"
-                android:layout_height="match_parent">
-
-                <TextView
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:gravity="end"
-                    android:text="@string/mqtt_uri_label" />
-
-                <TextView
-                    android:id="@+id/tv_mqtt_server_uri"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:gravity="center" />
-            </TableRow>
+            android:layout_height="match_parent">
+
+            <TextView
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:gravity="end"
+                android:text="@string/mqtt_uri_label" />
+
+            <TextView
+                android:id="@+id/tv_mqtt_server_uri"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:gravity="center" />
+        </TableRow>
+
+        <TableRow
+            android:layout_width="match_parent"
+            android:layout_height="match_parent">
 
-            <TableRow
-                android:layout_width="match_parent"
-                android:layout_height="match_parent">
+            <TextView
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:gravity="end"
+                android:text="@string/mqtt_state_label" />
 
-                <TextView
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:gravity="end"
-                    android:text="@string/mqtt_state_label" />
+            <TextView
+                android:id="@+id/tv_mqtt_state"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:gravity="center" />
 
-                <TextView
-                    android:id="@+id/tv_mqtt_state"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:gravity="center" />
+        </TableRow>
 
-            </TableRow>
-
-        </TableLayout>
-    </LinearLayout>
+    </TableLayout>
 
 </RelativeLayout>