From 6bbd09f8cef6361b646716501686e31faea67f88 Mon Sep 17 00:00:00 2001 From: Peter Edberg Date: Thu, 18 Jul 2024 15:22:25 -0700 Subject: [PATCH] CLDR-17812 v46 vxml: Fix TestCheckCLDR/TestPlaceholders, missing placeholder is OK for LOCALE_DEPENDENT --- .../unicode/cldr/unittest/TestCheckCLDR.java | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCheckCLDR.java b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCheckCLDR.java index 13ac7d2f6d1..caf5c34146e 100644 --- a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCheckCLDR.java +++ b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestCheckCLDR.java @@ -424,14 +424,16 @@ public void checkPlaceholders(CLDRFile cldrFileToTest) { } } else { // not disallowed if (!containsMessagePattern) { - errln( - cldrFileToTest.getLocaleID() - + " Value (" - + value - + ") contains placeholder, but placeholder info = «" - + placeholderStatus - + "»\t" - + path); + if (placeholderStatus != PlaceholderStatus.LOCALE_DEPENDENT) { + errln( + cldrFileToTest.getLocaleID() + + " Value (" + + value + + ") does not contain placeholder, but placeholder info = «" + + placeholderStatus + + "»\t" + + path); + } continue; } // get the set of placeholders