Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Unified Diff: source/test/intltest/alphaindextst.cpp

Issue 1621843002: ICU 56 update step 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu.git@561
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « source/test/intltest/Makefile.in ('k') | source/test/intltest/apicoll.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: source/test/intltest/alphaindextst.cpp
diff --git a/source/test/intltest/alphaindextst.cpp b/source/test/intltest/alphaindextst.cpp
index 847434704e0943b168047bda0ebe6fa68fc1a628..dd7984068d3f44bf7ca5d8734790a91986f7b813 100644
--- a/source/test/intltest/alphaindextst.cpp
+++ b/source/test/intltest/alphaindextst.cpp
@@ -1,6 +1,6 @@
/********************************************************************
* COPYRIGHT:
- * Copyright (c) 2012-2014, International Business Machines Corporation
+ * Copyright (c) 2012-2015, International Business Machines Corporation
* and others. All Rights Reserved.
********************************************************************/
//
@@ -494,6 +494,7 @@ static const char *localeAndIndexCharactersLists[][2] = {
/* English*/ {"en", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
/* Spanish*/ {"es", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:\\u00D1:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
/* Estonian*/ {"et", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:\\u0160:Z:\\u017D:T:U:V:\\u00D5:\\u00C4:\\u00D6:\\u00DC:X:Y"},
+ /* Basque*/ {"eu", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
/* Finnish*/ {"fi", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z:\\u00C5:\\u00C4:\\u00D6"},
/* Filipino*/ {"fil", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
/* French*/ {"fr", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
@@ -519,15 +520,6 @@ static const char *localeAndIndexCharactersLists[][2] = {
/* Vietnamese*/ {"vi", "A:\\u0102:\\u00C2:B:C:D:\\u0110:E:\\u00CA:F:G:H:I:J:K:L:M:N:O:\\u00D4:\\u01A0:P:Q:R:S:T:U:\\u01AF:V:W:X:Y:Z"},
/* Chinese*/ {"zh", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
/* Chinese (Traditional Han)*/ {"zh_Hant", "1\\u5283:2\\u5283:3\\u5283:4\\u5283:5\\u5283:6\\u5283:7\\u5283:8\\u5283:9\\u5283:10\\u5283:11\\u5283:12\\u5283:13\\u5283:14\\u5283:15\\u5283:16\\u5283:17\\u5283:18\\u5283:19\\u5283:20\\u5283:21\\u5283:22\\u5283:23\\u5283:24\\u5283:25\\u5283:26\\u5283:27\\u5283:28\\u5283:29\\u5283:30\\u5283:31\\u5283:32\\u5283:33\\u5283:35\\u5283:36\\u5283:39\\u5283:48\\u5283"},
-
- // As of ICU 52, ICU does not have collation data for the following language.
- // Therefore, constructing an AlphabeticIndex for it
- // ends up with a collator for the default locale
- // which makes the test unreliable. (see ticket #10277)
- // It exposes a bigger problem in that it may not be desirable for collation
- // to fall back to the default locale.
-
- // /* Basque*/ {"eu", "A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X:Y:Z"},
};
void AlphabeticIndexTest::TestIndexCharactersList() {
@@ -725,8 +717,9 @@ void AlphabeticIndexTest::TestChineseUnihan() {
// bucketIndex = radical number, adjusted for simplified radicals in lower buckets.
int32_t bucketIndex = index.getBucketIndex(UnicodeString((UChar)0x4e5d), status);
assertEquals("getBucketIndex(U+4E5D)", 5, bucketIndex);
+ // radical 100, and there is a 90' since Unicode 8
bucketIndex = index.getBucketIndex(UnicodeString((UChar)0x7527), status);
- assertEquals("getBucketIndex(U+7527)", 100, bucketIndex);
+ assertEquals("getBucketIndex(U+7527)", 101, bucketIndex);
}
#endif
« no previous file with comments | « source/test/intltest/Makefile.in ('k') | source/test/intltest/apicoll.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698