Minor tweak for getChildPaths : moved invariant out of loop

marcelmay · Feb 16, 2018 · 5b47c9e · 5b47c9e
1 parent 6016385
commit 5b47c9e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/src/main/java/de/m3y/hadoop/hdfs/hfsa/core/FSImageLoader.java b/lib/src/main/java/de/m3y/hadoop/hdfs/hfsa/core/FSImageLoader.java
@@ -474,10 +474,11 @@ public List<String> getChildPaths(String path) throws IOException {
         }
         long[] children = dirmap.get(rootNodeId);
         List<String> childPaths = new ArrayList<>();
+        final String pathWithTrailingSlash = ("/".equals(path) ? path : path + '/');
         for (long cid : children) {
             final FsImageProto.INodeSection.INode inode = fromINodeId(cid);
             if (inode.getType() == FsImageProto.INodeSection.INode.Type.DIRECTORY) {
-                childPaths.add(("/".equals(path) ? path : path + '/') + inode.getName().toStringUtf8());
+                childPaths.add(pathWithTrailingSlash + inode.getName().toStringUtf8());
             }
         }
         return childPaths;