Minor changes from code inspection results
authorsrowen <srowen@59b500cc-1b3d-0410-9834-0bbf25fbcc57>
Wed, 15 Sep 2010 09:21:06 +0000 (09:21 +0000)
committersrowen <srowen@59b500cc-1b3d-0410-9834-0bbf25fbcc57>
Wed, 15 Sep 2010 09:21:06 +0000 (09:21 +0000)
git-svn-id: http://zxing.googlecode.com/svn/trunk@1589 59b500cc-1b3d-0410-9834-0bbf25fbcc57

14 files changed:
android/src/com/google/zxing/client/android/CaptureActivityHandler.java
android/src/com/google/zxing/client/android/HelpActivity.java
android/src/com/google/zxing/client/android/book/SearchBookContentsAdapter.java
android/src/com/google/zxing/client/android/camera/CameraConfigurationManager.java
android/src/com/google/zxing/client/android/camera/FlashlightManager.java
android/src/com/google/zxing/client/android/encode/QRCodeEncoder.java
android/src/com/google/zxing/client/android/result/AddressBookResultHandler.java
android/src/com/google/zxing/client/android/result/CalendarResultHandler.java
android/src/com/google/zxing/client/android/result/ResultHandler.java
android/src/com/google/zxing/client/android/result/SMSResultHandler.java
android/src/com/google/zxing/client/android/result/WifiResultHandler.java
android/src/com/google/zxing/client/android/share/AppPickerActivity.java
android/src/com/google/zxing/client/android/share/LoadPackagesAsyncTask.java
android/src/com/google/zxing/client/android/wifi/WifiReceiver.java

index 46ee8a8..525e83c 100755 (executable)
@@ -114,6 +114,7 @@ public final class CaptureActivityHandler extends Handler {
     try {
       decodeThread.join();
     } catch (InterruptedException e) {
+      // continue
     }
 
     // Be absolutely sure we don't send any queued up messages
index f658d88..34d80ce 100644 (file)
@@ -36,6 +36,7 @@ import android.widget.Button;
  * @author dswitkin@google.com (Daniel Switkin)
  */
 public final class HelpActivity extends Activity {
+
   private static final String TAG = HelpActivity.class.getSimpleName();
 
   // Actually guessing at the Desire's MODEL for now:
@@ -107,9 +108,9 @@ public final class HelpActivity extends Activity {
       webView.loadUrl(BASE_URL + DEFAULT_PAGE);
     }
 
-    backButton = (Button)findViewById(R.id.back_button);
+    backButton = (Button) findViewById(R.id.back_button);
     backButton.setOnClickListener(backListener);
-    Button doneButton = (Button)findViewById(R.id.done_button);
+    View doneButton = findViewById(R.id.done_button);
     doneButton.setOnClickListener(doneListener);
 
     if (!initialized) {
index 32fbb10..c8d4122 100644 (file)
@@ -32,7 +32,8 @@ import com.google.zxing.client.android.R;
  * @author dswitkin@google.com (Daniel Switkin)
  */
 final class SearchBookContentsAdapter extends ArrayAdapter<SearchBookContentsResult> {
-  public SearchBookContentsAdapter(Context context, List<SearchBookContentsResult> items) {
+
+  SearchBookContentsAdapter(Context context, List<SearchBookContentsResult> items) {
     super(context, R.layout.search_book_contents_list_item, 0, items);
   }
 
index 6e6cd8e..071388f 100644 (file)
@@ -118,8 +118,7 @@ final class CameraConfigurationManager {
     return cameraResolution;
   }
 
-  private static Point findBestPreviewSizeValue(String previewSizeValueString,
-      Point screenResolution) {
+  private static Point findBestPreviewSizeValue(CharSequence previewSizeValueString, Point screenResolution) {
     int bestX = 0;
     int bestY = 0;
     int diff = Integer.MAX_VALUE;
index 547d98d..c7f9541 100644 (file)
@@ -38,8 +38,8 @@ final class FlashlightManager {
 
   private static final String TAG = FlashlightManager.class.getSimpleName();
 
-  private static Object iHardwareService;
-  private static Method setFlashEnabledMethod;
+  private static final Object iHardwareService;
+  private static final Method setFlashEnabledMethod;
   static {
     iHardwareService = getHardwareService();
     setFlashEnabledMethod = getSetFlashEnabledMethod(iHardwareService);
index 393967e..873ccd5 100755 (executable)
@@ -204,8 +204,8 @@ final class QRCodeEncoder {
     } else if (type.equals(Contents.Type.CONTACT)) {
       Bundle bundle = intent.getBundleExtra(Intents.Encode.DATA);
       if (bundle != null) {
-        StringBuilder newContents = new StringBuilder();
-        StringBuilder newDisplayContents = new StringBuilder();
+        StringBuilder newContents = new StringBuilder(100);
+        StringBuilder newDisplayContents = new StringBuilder(100);
         newContents.append("MECARD:");
         String name = trim(bundle.getString(Contacts.Intents.Insert.NAME));
         if (name != null) {
@@ -258,8 +258,8 @@ final class QRCodeEncoder {
   }
 
   private boolean encodeQRCodeContents(AddressBookParsedResult contact) {
-    StringBuilder newContents = new StringBuilder();
-    StringBuilder newDisplayContents = new StringBuilder();
+    StringBuilder newContents = new StringBuilder(100);
+    StringBuilder newDisplayContents = new StringBuilder(100);
     newContents.append("MECARD:");
     String[] names = contact.getNames();
     if (names != null && names.length > 0) {
@@ -322,10 +322,10 @@ final class QRCodeEncoder {
                                BarcodeFormat format,
                                int desiredWidth,
                                int desiredHeight) throws WriterException {
-    Hashtable hints = null;
+    Hashtable<EncodeHintType,Object> hints = null;
     String encoding = guessAppropriateEncoding(contents);
     if (encoding != null) {
-      hints = new Hashtable(2);
+      hints = new Hashtable<EncodeHintType,Object>(2);
       hints.put(EncodeHintType.CHARACTER_SET, encoding);
     }
     MultiFormatWriter writer = new MultiFormatWriter();    
index 004abd5..ece7f81 100644 (file)
@@ -157,7 +157,7 @@ public final class AddressBookResultHandler extends ResultHandler {
   @Override
   public CharSequence getDisplayContents() {
     AddressBookParsedResult result = (AddressBookParsedResult) getResult();
-    StringBuffer contents = new StringBuffer();
+    StringBuffer contents = new StringBuffer(100);
     ParsedResult.maybeAppend(result.getNames(), contents);
     int namesLength = contents.length();
 
index 76c9cd3..7d522d3 100644 (file)
@@ -72,7 +72,7 @@ public final class CalendarResultHandler extends ResultHandler {
   @Override
   public CharSequence getDisplayContents() {
     CalendarParsedResult calResult = (CalendarParsedResult) getResult();
-    StringBuffer result = new StringBuffer();
+    StringBuffer result = new StringBuffer(100);
     ParsedResult.maybeAppend(calResult.getSummary(), result);
     appendTime(calResult.getStart(), result);
 
index 5c88165..cc4aac4 100644 (file)
@@ -40,7 +40,6 @@ import android.net.Uri;
 import android.preference.PreferenceManager;
 import android.provider.Contacts;
 import android.view.View;
-import android.widget.Button;
 
 import java.text.DateFormat;
 import java.text.ParsePosition;
@@ -97,7 +96,7 @@ public abstract class ResultHandler {
 
     // Make sure the Shopper button is hidden by default. Without this, scanning a product followed
     // by a QR Code would leave the button on screen among the QR Code actions.
-    Button shopperButton = (Button) activity.findViewById(R.id.shopper_button);
+    View shopperButton = activity.findViewById(R.id.shopper_button);
     shopperButton.setVisibility(View.GONE);
   }
 
@@ -138,7 +137,7 @@ public abstract class ResultHandler {
    * @param listener The on click listener to install for this button.
    */
   protected void showGoogleShopperButton(View.OnClickListener listener) {
-    Button shopperButton = (Button) activity.findViewById(R.id.shopper_button);
+    View shopperButton = activity.findViewById(R.id.shopper_button);
     shopperButton.setVisibility(View.VISIBLE);
     shopperButton.setOnClickListener(listener);
   }
index 3675e7a..ef57fb1 100644 (file)
@@ -65,7 +65,7 @@ public final class SMSResultHandler extends ResultHandler {
   @Override
   public CharSequence getDisplayContents() {
     SMSParsedResult smsResult = (SMSParsedResult) getResult();
-    StringBuffer contents = new StringBuffer();
+    StringBuffer contents = new StringBuffer(50);
     String[] rawNumbers = smsResult.getNumbers();
     String[] formattedNumbers = new String[rawNumbers.length];
     for (int i = 0; i < rawNumbers.length; i++) {
index 9372dbb..f22533a 100644 (file)
@@ -62,7 +62,7 @@ public final class WifiResultHandler extends ResultHandler {
   @Override
   public CharSequence getDisplayContents() {
     WifiParsedResult wifiResult = (WifiParsedResult) getResult();
-    StringBuffer contents = new StringBuffer();
+    StringBuffer contents = new StringBuffer(50);
     String wifiLabel = parent.getString(R.string.wifi_ssid_label);
     ParsedResult.maybeAppend(wifiLabel + '\n' + wifiResult.getSsid(), contents);
     String typeLabel = parent.getString(R.string.wifi_type_label);
index 6470919..56ad642 100644 (file)
@@ -18,6 +18,7 @@ package com.google.zxing.client.android.share;
 
 import android.app.ListActivity;
 import android.app.ProgressDialog;
+import android.content.DialogInterface;
 import android.content.Intent;
 import android.os.Bundle;
 import android.provider.Browser;
@@ -31,7 +32,7 @@ import java.util.List;
 public final class AppPickerActivity extends ListActivity {
 
   private final List<String[]> labelsPackages = new ArrayList<String[]>();
-  private ProgressDialog dialog;
+  private DialogInterface dialog;
 
   @Override
   protected void onCreate(Bundle icicle) {
@@ -58,7 +59,7 @@ public final class AppPickerActivity extends ListActivity {
     finish();
   }
 
-  ProgressDialog getProgressDialog() {
+  DialogInterface getProgressDialog() {
     return dialog;
   }
 
index 7f17c13..9e29378 100644 (file)
@@ -22,6 +22,7 @@ import android.os.AsyncTask;
 import android.widget.ArrayAdapter;
 import android.widget.ListAdapter;
 
+import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Comparator;
@@ -65,11 +66,7 @@ final class LoadPackagesAsyncTask extends AsyncTask<List<String[]>,Void,List<Str
         }
       }
     }
-    Collections.sort(labelsPackages, new Comparator<String[]>() {
-      public int compare(String[] o1, String[] o2) {
-        return o1[0].compareTo(o2[0]);
-      }
-    });
+    Collections.sort(labelsPackages, new ByFirstStringComparator());
     return labelsPackages;
   }
 
@@ -102,4 +99,10 @@ final class LoadPackagesAsyncTask extends AsyncTask<List<String[]>,Void,List<Str
     appPickerActivity.getProgressDialog().dismiss();
   }
 
+  private static class ByFirstStringComparator implements Comparator<String[]>, Serializable {
+    public int compare(String[] o1, String[] o2) {
+      return o1[0].compareTo(o2[0]);
+    }
+  }
+
 }
index 03ee947..de0f703 100644 (file)
@@ -70,7 +70,7 @@ final class WifiReceiver extends BroadcastReceiver {
           }
           if (state == NetworkInfo.State.DISCONNECTED){
             Log.d(TAG, "Got state: " + state + " for ssid: " + ssid);
-            ((WifiActivity)parent).gotError();
+            parent.gotError();
           }
         }
       }