Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLDR-17217 Reserve datetime fields for private use #3603

Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
CLDR-17217 Reserve datetime fields for private use
macchiati committed Apr 2, 2024
commit c59ccd47f5f59542a49ed88678db1a656bec108c
1 change: 1 addition & 0 deletions docs/ldml/tr35-dates.md
Original file line number Diff line number Diff line change
@@ -2002,6 +2002,7 @@ The Date Field Symbol Table below shows the pattern characters (Sym.) and associ
<tr><td>3</td><td>Abbreviated (e.g. MMM)</td></tr>
<tr><td>4</td><td colspan="2">Wide / Long / Full (e.g. MMMM, EEEE)</td></tr>
<tr><td>5</td><td colspan="2">Narrow (e.g. MMMMM, EEEEE)<br/>(The counter-intuitive use of 5 letters for this is forced by backwards compatibility)</td></tr>
<tr><td>&gt;16</td><td colspan="2">Private Use<br/>(Reserved for use by implementations using CLDR; will never be otherwise used by CLDR.)</td></tr>
</table>

Notes for the table below:

Unchanged files with check annotations Beta

}
if (!collectedBad.isEmpty()) {
warnln(

Check warning on line 539 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/LikelySubtagsTest.java

GitHub Actions / build

(LikelySubtagsTest.java:539) Warning: Locales have 224 unexpected characters in main and/or aux: [҂״܀-܍०-९৲-৺੦-੯૰౦-౯೦-೯໐-໙၀-၉၏႐-႙႞႟፠-፼᎐-᎙᠐-᠙꒐-꓆𞅏{a\:}{ch’}{e\:}{i\:}{k’}{o\:}{ts’}{tł’}{t’}{à\:}{á\:}{è\:}{é\:}{ì\:}{í\:}{ò\:}{ó\:}{ଅ\:}{ଆ\:}{ଏ\:}]
"Locales have "
+ collectedBad.size()
+ " unexpected characters in main and/or aux:\t"
// SUPPLEMENTAL_DATA_INFO.getLikelyOrigins().get(value);
// fieldToOrigin.put(value, origin == null ? "n/a" : origin);
// }
warnln("Bad status=" + entry.getKey() + " for " + entry.getValue());

Check warning on line 784 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/LikelySubtagsTest.java

GitHub Actions / build

(LikelySubtagsTest.java:784) Warning: Bad status=macroregion for [001, 419]
}
}
}
if (logKnownIssue("CLDR-16394", "slightly out of order")) {
errorType = WARN;
}
msg(

Check warning on line 440 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestAnnotations.java

GitHub Actions / build

(TestAnnotations.java:440) Warning: Out of order: 🧔‍♀ (57500) woman: beard > 👩 (61000) woman

Check warning on line 440 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestAnnotations.java

GitHub Actions / build

(TestAnnotations.java:440) Warning: Out of order: 👱‍♀ (66500) woman: blond hair > 👱‍♂ (67700) man: blond hair

Check warning on line 440 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestAnnotations.java

GitHub Actions / build

(TestAnnotations.java:440) Warning: Out of order: 👣 (339200) footprints > 🏻 (339300) light skin tone
"Out of order: "
+ lastEmoji
+ " ("
void show(boolean verbose, ImmutableSet<ValueStatus> retain) {
if (dtdData.dtdType == DtdType.keyboard3
&& testLog.logKnownIssue("CLDR-14974", "skipping for keyboard")) {
testLog.warnln("Skipping for keyboard3");

Check warning on line 308 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestAttributeValues.java

GitHub Actions / build

(TestAttributeValues.java:308) Warning: Skipping for keyboard3
}
boolean haveProblems = false;
for (ValueStatus valueStatus : ValueStatus.values()) {
"blo", "eo", "ie", "kxv", "lij", "lmo", "nds", "prg", "szl", "tok", "vec",
"vmw", "xnr", "za");
warnln(

Check warning on line 254 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java

GitHub Actions / build

(TestCLDRLocaleCoverage.java:254) Warning: Locale names added for translation; revisit each release:
"Locale names added for translation; revisit each release:\n"
+ Joiner.on("\n")
.join(
coverageLocales, localesForNames);
if (showRegex || true) {
String simplePattern = MinimizeRegex.simplePattern(localesForNames);
warnln("Plain Regex for coverage:\n" + simplePattern);

Check warning on line 331 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java

GitHub Actions / build

(TestCLDRLocaleCoverage.java:331) Warning: Plain Regex for coverage:
warnln(

Check warning on line 332 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java

GitHub Actions / build

(TestCLDRLocaleCoverage.java:332) Warning: Compact Regex for coverage:
"Compact Regex for coverage:\n"
+ MinimizeRegex.compressWith(localesForNames, new UnicodeSet("[a-z]")));
}
official1MSet.add(locale);
}
}
warnln("Official with 1M+ speakers, need investigation of literacy: " + official1MSet);

Check warning on line 346 in tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCLDRLocaleCoverage.java

GitHub Actions / build

(TestCLDRLocaleCoverage.java:346) Warning: Official with 1M+ speakers, need investigation of literacy: [ak, bo, ckb, ee, ff, gsw, kg, ln, mdh, mg, nb, nd, rn, rw, sg, sn, tsg, vec, za]
// assertContains("sdiLocales contains oldModernLocales", sdiLocales,
// oldModernLocales);