Apply suggestions from code review

Co-authored-by: Kezhu Wang <[email protected]>
apache · Dec 19, 2024 · 31bcdea · 31bcdea
1 parent 44f3198
commit 31bcdea
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/...tor-framework/src/main/java/org/apache/curator/framework/imps/GetChildrenBuilderImpl.java b/...tor-framework/src/main/java/org/apache/curator/framework/imps/GetChildrenBuilderImpl.java
@@ -165,14 +165,11 @@ public void performBackgroundOperation(final OperationAndData<String> operationA
                 @Override
                 public void processResult(int rc, String path, Object o, List<String> strings, Stat stat) {
                     watching.commitWatcher(rc, false);
-                    if (strings == null) {
-                        strings = Lists.newArrayList();
-                    }
                     trace.setReturnCode(rc)
                             .setPath(path)
                             .setWithWatcher(watching.hasWatcher())
                             .setStat(stat)
-                            .setResponseChildrenCount(strings.size())
+                            .setResponseChildrenCount(strings != null ? strings.size() : 0)
                             .commit();
                     CuratorEventImpl event = new CuratorEventImpl(
                             client,