diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2024-04-11 12:15:30 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2024-04-11 12:15:30 +0200 |
commit | c08b60a611e71e50db460070f4f9f441abf44d76 (patch) | |
tree | 1da522a06b091cf3e6c8394363276db3c0af2d04 /container-search | |
parent | 5a4aa7df8f19d6672779f6d1ce78459e38e4d811 (diff) |
Use Set.of() gives a set where contains check if argument is null
Diffstat (limited to 'container-search')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/result/Hit.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/result/Hit.java b/container-search/src/main/java/com/yahoo/search/result/Hit.java index 045388ba31c..d0bc80b6950 100644 --- a/container-search/src/main/java/com/yahoo/search/result/Hit.java +++ b/container-search/src/main/java/com/yahoo/search/result/Hit.java @@ -333,7 +333,8 @@ public class Hit extends ListenableFreezableClass implements Data, Comparable<Hi */ public void setFillable() { if (filled == null) { - filled = Set.of(); + //TODO Should have used Set.of(), but it checks that contains it not called with null + filled = Collections.emptySet(); unmodifiableFilled = filled; } } @@ -347,7 +348,8 @@ public class Hit extends ListenableFreezableClass implements Data, Comparable<Hi */ public void setFilled(String summaryClass) { if (filled == null || filled.isEmpty()) { - filled = Set.of(summaryClass); + //TODO Should have used Set.of(), but it checks that contains it not called with null + filled = Collections.singleton(summaryClass); unmodifiableFilled = filled; } else if (filled.size() == 1) { filled = new HashSet<>(filled); |