findbugs fixit: misc cleanup in CallCard.java:
David Brown [Thu, 14 May 2009 21:43:28 +0000 (14:43 -0700)]
  - remove an unused variation of updateDisplayForPerson()
  - fix potential NPE when calling Connection.getNumberPresentation()
  - remove an unused member variable
  - remove a couple of unused local variables

src/com/android/phone/CallCard.java

index 3ac838d..7480e13 100644 (file)
@@ -91,8 +91,6 @@ public class CallCard extends FrameLayout
     // Menu button hint
     private TextView mMenuButtonHint;
 
-    private boolean mRingerSilenced;
-
     private CallTime mCallTime;
 
     // Track the state for the photo.
@@ -135,8 +133,6 @@ public class CallCard extends FrameLayout
     void reset() {
         if (DBG) log("reset()...");
 
-        mRingerSilenced = false;
-
         // default to show ACTIVE call style, with empty title and status text
         showCallConnected();
         setUpperTitle("");
@@ -159,8 +155,6 @@ public class CallCard extends FrameLayout
 
         if (DBG) log("CallCard onFinishInflate(this = " + this + ")...");
 
-        LayoutInflater inflater = LayoutInflater.from(getContext());
-
         mMainCallCard = (ViewGroup) findViewById(R.id.mainCallCard);
         mOtherCallOngoingInfoArea = (ViewGroup) findViewById(R.id.otherCallOngoingInfoArea);
         mOtherCallOnHoldInfoArea = (ViewGroup) findViewById(R.id.otherCallOnHoldInfoArea);
@@ -455,18 +449,17 @@ public class CallCard extends FrameLayout
             // has only one connection.)
             Connection conn = call.getEarliestConnection();
 
-            int presentation = conn.getNumberPresentation(); 
-
             if (conn == null) {
                 if (DBG) log("displayMainCallStatus: connection is null, using default values.");
                 // if the connection is null, we run through the behaviour
                 // we had in the past, which breaks down into trivial steps
                 // with the current implementation of getCallerInfo and
                 // updateDisplayForPerson.
-                CallerInfo info = PhoneUtils.getCallerInfo(getContext(), conn);
-                updateDisplayForPerson(info, presentation, false, call);
+                CallerInfo info = PhoneUtils.getCallerInfo(getContext(), null /* conn */);
+                updateDisplayForPerson(info, Connection.PRESENTATION_ALLOWED, false, call);
             } else {
                 if (DBG) log("  - CONN: " + conn + ", state = " + conn.getState());
+                int presentation = conn.getNumberPresentation();
 
                 // make sure that we only make a new query when the current
                 // callerinfo differs from what we've been requested to display.
@@ -809,7 +802,6 @@ public class CallCard extends FrameLayout
 
 
     private String getCallFailedString(Call call) {
-        Phone phone = PhoneApp.getInstance().phone;
         Connection c = call.getEarliestConnection();
         int resID;
 
@@ -888,15 +880,6 @@ public class CallCard extends FrameLayout
     }
 
     /**
-     *  Add the Call object to these next 2 apis since the callbacks from
-     *  updateImageViewWithContactPhotoAsync call will need to use it.
-     */
-
-    private void updateDisplayForPerson(CallerInfo info, int presentation, Call call) {
-        updateDisplayForPerson(info, presentation, false, call);
-    }
-
-    /**
      * Updates the name / photo / number / label fields on the CallCard
      * based on the specified CallerInfo.
      *