diff options
author | Yorke Lee <yorkelee@google.com> | 2013-11-18 10:03:59 -0800 |
---|---|---|
committer | Yorke Lee <yorkelee@google.com> | 2013-11-18 10:03:59 -0800 |
commit | dbe9db9e26d4c7ca08ba24630e990e25629aac78 (patch) | |
tree | 5b5029f33e7249b94989927786dfe7cbea12658d | |
parent | 9acb00ee1f2fd8cb190af290a6fe12e01e0d99c1 (diff) |
Use single instance of PhoneNumberUtilsWrapper
PhoneNumberUtilsWrapper is creating a new instance of the class
everytime it tries to look up an unknown number. Just use a single
instance for this instead to avoid redundant object creation.
Change-Id: I418d95ced42c1db06c63848768050a181defb347
-rw-r--r-- | src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java b/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java index 32b49b336..242be2d04 100644 --- a/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java +++ b/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java @@ -28,7 +28,7 @@ import java.util.Set; * */ public class PhoneNumberUtilsWrapper { - + private static final PhoneNumberUtilsWrapper INSTANCE = new PhoneNumberUtilsWrapper(); private static final Set<String> LEGACY_UNKNOWN_NUMBERS = Sets.newHashSet("-1", "-2", "-3"); /** Returns true if it is possible to place a call to the given number. */ @@ -74,7 +74,7 @@ public class PhoneNumberUtilsWrapper { if (TextUtils.isEmpty(number)) { return false; } - if (new PhoneNumberUtilsWrapper().isVoicemailNumber(number)) { + if (INSTANCE.isVoicemailNumber(number)) { return false; } if (isLegacyUnknownNumbers(number.toString())) { |