commit 46befc3c110616a775b03fdb51778037900f457e
author Daisuke Miyakawa <dmiyakawa@google.com> Wed Oct 28 10:13:57 2009 +0900
committer Daisuke Miyakawa <dmiyakawa@google.com> Wed Oct 28 10:13:57 2009 +0900
tree a540c24ed49d1a2206ffa3e33c6bb0e6db7bef64
parent 951fa9c0c55ba87d18b92a77e4d1d8091b29d7c9

Consolidate Contacts code.

As per the issue 2217164, I roughly looked over how the variable "phoneNumber" is treaded in Eclair, and found the code around it looked a little flaky.

I believe this change does not make Contacts code easier to read, while there's no malicious regression.

Internal issue number: 2217164

src/com/android/contacts/PhoneDisambigDialog.java

diff --git a/src/com/android/contacts/PhoneDisambigDialog.java b/src/com/android/contacts/PhoneDisambigDialog.java
index b727c77..e0295e3 100644
--- a/src/com/android/contacts/PhoneDisambigDialog.java
+++ b/src/com/android/contacts/PhoneDisambigDialog.java
@@ -136,11 +136,11 @@
 
     private class PhoneItem implements Collapsible<PhoneItem> {
 
-        String phoneNumber;
-        long id;
+        final String phoneNumber;
+        final long id;
 
         public PhoneItem(String newPhoneNumber, long newId) {
-            phoneNumber = newPhoneNumber;
+            phoneNumber = (newPhoneNumber != null ? newPhoneNumber : "");
             id = newId;
         }
 
@@ -160,6 +160,7 @@
             return false;
         }
 
+        @Override
         public String toString() {
             return phoneNumber;
         }