findbugs fixit: Clean up a few more (fairly minor) findbugs complaints.
David Brown [Thu, 14 May 2009 22:39:18 +0000 (15:39 -0700)]
src/com/android/phone/CallNotifier.java
src/com/android/phone/DTMFTwelveKeyDialer.java
src/com/android/phone/DTMFTwelveKeyDialerView.java
src/com/android/phone/EmergencyDialer.java
src/com/android/phone/InCallScreen.java
src/com/android/phone/OutgoingCallBroadcaster.java

index da49d77..e107872 100644 (file)
@@ -186,7 +186,6 @@ public class CallNotifier extends Handler
     private void onNewRingingConnection(AsyncResult r) {
         Connection c = (Connection) r.result;
         if (DBG) log("onNewRingingConnection(): " + c);
-        PhoneApp app = PhoneApp.getInstance();
 
         // Incoming calls are totally ignored if the device isn't provisioned yet
         boolean provisioned = Settings.Secure.getInt(mPhone.getContext().getContentResolver(),
@@ -618,7 +617,7 @@ public class CallNotifier extends Handler
                 // the user deliberately rejected.)
 
                 PhoneUtils.CallerInfoToken info =
-                    PhoneUtils.startGetCallerInfo(mApplication, c, this, new Long(date));
+                        PhoneUtils.startGetCallerInfo(mApplication, c, this, Long.valueOf(date));
                 if (info != null) {
                     // at this point, we've requested to start a query, but it makes no
                     // sense to log this missed call until the query comes back.
index 667d6e7..31b6cd0 100644 (file)
@@ -61,7 +61,7 @@ public class DTMFTwelveKeyDialer implements
     // events
     private static final int PHONE_DISCONNECT = 100;
 
-    private static Phone mPhone;
+    private Phone mPhone;
     private ToneGenerator mToneGenerator;
     private Object mToneGeneratorLock = new Object();
 
@@ -130,7 +130,7 @@ public class DTMFTwelveKeyDialer implements
      * Create an input method just so that the textview can display the cursor.
      * There is no selecting / positioning on the dialer field, only number input.
      */
-    private class DTMFDisplayMovementMethod implements MovementMethod {
+    private static class DTMFDisplayMovementMethod implements MovementMethod {
 
         /**Return false since we are NOT consuming the input.*/
         public boolean onKeyDown(TextView widget, Spannable buffer, int keyCode, KeyEvent event) {
index e8e72e6..910db76 100644 (file)
@@ -91,7 +91,6 @@ class DTMFTwelveKeyDialerView extends LinearLayout {
 
         final int numTouchables = touchables.size();
         
-        Rect closestBounds = new Rect();
         Rect touchableBounds = new Rect();
         
         for (int i = 0; i < numTouchables; i++) {
index cf20ef7..599d83c 100644 (file)
@@ -286,8 +286,6 @@ View.OnLongClickListener, View.OnKeyListener, TextWatcher {
     }
 
     public void onClick(View view) {
-        final Editable digits = mDigits.getText();
-
         switch (view.getId()) {
             case R.id.one: {
                 playTone(ToneGenerator.TONE_DTMF_1);
index bce7625..735b2f9 100644 (file)
@@ -2158,7 +2158,6 @@ public class InCallScreen extends Activity
         // connections still in that state.]
         mPhone.clearDisconnected();
 
-        final PhoneApp app = PhoneApp.getInstance();
         if (!phoneIsInUse()) {
             // Phone is idle!  We should exit this screen now.
             if (DBG) log("- delayedCleanupAfterDisconnect: phone is idle...");
index 99ed6af..f895de2 100644 (file)
@@ -45,14 +45,10 @@ public class OutgoingCallBroadcaster extends Activity {
     public static final String EXTRA_ALREADY_CALLED = "android.phone.extra.ALREADY_CALLED";
     public static final String EXTRA_ORIGINAL_URI = "android.phone.extra.ORIGINAL_URI";
 
-    private Phone mPhone;
-
     @Override
     protected void onCreate(Bundle icicle) {
         super.onCreate(icicle);
 
-        mPhone = PhoneApp.getInstance().phone;
-
         Intent intent = getIntent();
         if (LOGV) Log.v(TAG, "onResume: Got intent " + intent + ".");