> On 18 Aug 2016, at 16:36, Steve Drach <[email protected]> wrote:
>
> Hi,
>
> Please review this rather simple fix to JarFileSystem.
>
> issue: https://bugs.openjdk.java.net/browse/JDK-8164389
> <https://bugs.openjdk.java.net/browse/JDK-8164389>
> webrev: http://cr.openjdk.java.net/~sdrach/8164389/webrev/index.html
> <http://cr.openjdk.java.net/~sdrach/8164389/webrev/index.html>
>
JarFileSystem
—
162 private void walk(IndexNode inode, Consumer<IndexNode> process) {
163 if (inode == null) return;
164 if (inode.isDir())
165 walk(inode.child, process);
166 else
167 process.accept(inode);
168 walk(inode.sibling, process);
169 }
I would recommend using braces, even for single statement blocks.
JFSTester
—
You might wanna create a temporary jar file if possible, just in case the test
somehow fails to clean things up.
80 public void TestWalk() throws IOException {
Lower case for first letter.
Paul.
