Fix getSet on properties of strings when using incremental properties

unicode-org · Mar 26, 2024 · f11eb25 · f11eb25
1 parent ecec2fc
commit f11eb25
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/unicodetools/src/main/java/org/unicode/props/IndexUnicodeProperties.java b/unicodetools/src/main/java/org/unicode/props/IndexUnicodeProperties.java
@@ -804,6 +804,14 @@ public UnicodeSet getSet(PatternMatcher matcher, UnicodeSet result) {
                 System.out.println(
                         "Long getSet for U" + ucdVersion + ":" + prop + " (" + Δt_in_ms + " ms)");
             }
+            // We only do the delta thing for code points; for strings, we need to do the lookup
+            // directly (and clean whatever was added by walking through history).
+            if (baseVersionProperties != null
+                    && (result.hasStrings()
+                            || (_getRawUnicodeMap().stringKeys() != null
+                                    && !_getRawUnicodeMap().stringKeys().isEmpty()))) {
+                result.removeAllStrings().addAll(super.getSet(matcher, new UnicodeSet()).strings());
+            }
             return result;
         }