Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge change 26125 into eclair

Browse Source 
* changes:
  make PhoneNumberUtils.compareLoosely use MIN_MATCH of 7 instead of 5.
This commit is contained in:
Android (Google) Code Review
2009-09-21 12:59:57 -04:00
parent 483a599c03 02b5948a43
commit 5ef75b68fe
1 changed files with 19 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
telephony/java/android/telephony/PhoneNumberUtils.java
View File

@@ -317,7 +317,7 @@ public class PhoneNumberUtils
* enough for caller ID purposes. * enough for caller ID purposes.
* *
* - Compares from right to left * - Compares from right to left
* - requires MIN_MATCH (5) characters to match * - requires MIN_MATCH (7) characters to match
* - handles common trunk prefixes and international prefixes * - handles common trunk prefixes and international prefixes
* (basically, everything except the Russian trunk prefix) * (basically, everything except the Russian trunk prefix)
* *
@@ -1202,10 +1202,24 @@ public class PhoneNumberUtils
JapanesePhoneNumberFormatter.format(text); JapanesePhoneNumberFormatter.format(text);
} }
// Three and four digit phone numbers for either special services // Three and four digit phone numbers for either special services,
// or from the network (eg carrier-originated SMS messages) should // or 3-6 digit addresses from the network (eg carrier-originated SMS messages) should
// not match // not match.
static final int MIN_MATCH = 5; //
// This constant used to be 5, but SMS short codes has increased in length and
// can be easily 6 digits now days. Most countries have SMS short code length between
// 3 to 6 digits. The exceptions are
//
// Australia: Short codes are six or eight digits in length, starting with the prefix "19"
// followed by an additional four or six digits and two.
// Czech Republic: Codes are seven digits in length for MO and five (not billed) or
// eight (billed) for MT direction
//
// see http://en.wikipedia.org/wiki/Short_code#Regional_differences for reference
//
// However, in order to loose match 650-555-1212 and 555-1212, we need to set the min match
// to 7.
static final int MIN_MATCH = 7;
/** /**
* isEmergencyNumber: checks a given number against the list of * isEmergencyNumber: checks a given number against the list of