Skip to content

Commit 530111f

Browse files
authored
CLDR-18852 Gujr transform (#5014)
1 parent 47a2848 commit 530111f

File tree

4 files changed

+10
-13
lines changed

4 files changed

+10
-13
lines changed

common/transforms/Gujarati-Latin.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ For terms of use, see http://www.unicode.org/copyright.html
1010
<transforms>
1111
<transform source="Gujr" target="Latn" direction="forward" alias="Gujarati-Latin und-Latn-t-und-gujr">
1212
<tRule>
13-
::[।-॥ઁ-ઃઅ-ઍએ-ઑઓ-નપ-રલ-ળવ-હ઼-ૅે-ૉો-્ૐૠૡ૦-૯];
13+
::[।-॥ઁ-ઃઅ-ઍએ-ઑઓ-નપ-રલ-ળવ-હ઼-ૅે-ૉો-્ૐૠૡ૦-૯];
1414
::NFD;
1515
::Gujarati-InterIndic;
1616
::InterIndic-Latin;

tools/cldr-code/src/main/java/org/unicode/cldr/util/CLDRTransforms.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -957,7 +957,7 @@ public static void testSourceTarget(Transliterator t, String... sourcePairs) {
957957

958958
/**
959959
* Gets a transform from a script to Latin. for testing For a locale, use
960-
* ExemplarUtilities.getScript(locale) to get the script
960+
* ExemplarUtilities.getLikelyScript(locale) to get the script
961961
*/
962962
public static Transliterator getTestingLatinScriptTransform(final String script) {
963963
String id;

tools/cldr-code/src/main/java/org/unicode/cldr/util/ExemplarUtilities.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ public class ExemplarUtilities {
99
static LikelySubtags ls = new LikelySubtags();
1010
static LanguageTagParser ltp = new LanguageTagParser();
1111

12-
public static synchronized String getScript(String locale) {
12+
public static synchronized String getLikelyScript(String locale) {
1313
String max = ls.maximize(locale);
1414
return ltp.set(max).getScript();
1515
}

tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestTransforms.java

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -900,12 +900,10 @@ public void TestLocales() {
900900
if (!ltp.set(locale).getRegion().isEmpty()) {
901901
continue;
902902
}
903-
String max = ls.maximize(locale);
904-
final String script = ltp.set(max).getScript();
903+
final String script = ExemplarUtilities.getLikelyScript(locale);
905904
if (script.equals("Latn")) {
906905
continue;
907906
}
908-
909907
Transliterator t;
910908
try {
911909
t = CLDRTransforms.getTestingLatinScriptTransform(script);
@@ -928,7 +926,7 @@ public void TestLocales() {
928926
locale
929927
+ " "
930928
+ script
931-
+ " transform doesn't handle "
929+
+ "-Latin transform doesn't handle "
932930
+ badPlusSample.size()
933931
+ " code points:\n"
934932
+ badPlusSample);
@@ -964,11 +962,6 @@ public void TestLocales() {
964962
for (String s : suppressHack) {
965963
allMissing.scriptMissing.remove(s);
966964
}
967-
if (allMissing.scriptMissing.containsKey("\u0AF0")
968-
&& logKnownIssue("CLDR-18852", "U+0AF0 missing in Gujr translit")) {
969-
allMissing.scriptMissing.remove("\u0AF0");
970-
}
971-
972965
for (String script : allMissing.scriptMissing.values()) {
973966
UnicodeSet missingFoScript = allMissing.scriptMissing.getKeys(script);
974967
String localeForScript =
@@ -1012,7 +1005,11 @@ public SampleData(String locale, String path, String value, String transformed)
10121005
public String toString() {
10131006
return String.format(
10141007
"%s;\t%s;\t%s;\t%s;\t%s",
1015-
locale, ExemplarUtilities.getScript(locale), path, value, transformed);
1008+
locale,
1009+
ExemplarUtilities.getLikelyScript(locale),
1010+
path,
1011+
value,
1012+
transformed);
10161013
}
10171014
}
10181015

0 commit comments

Comments
 (0)