RFR: Fix NullPointerException on a parallel run [v5]

Dmitry Bessonov dbessono at openjdk.org
Thu Sep 1 12:19:27 UTC 2022


On Thu, 1 Sep 2022 12:12:12 GMT, Jan Kratochvil <duke at openjdk.org> wrote:

> It won't generate NPE but it will produce `String "null/filename"`. Which will produce NPE later in the code like the original problem this patch tries to fix. But then we do not know what return value `null` from `getRootRelativePath()` should mean. So we do not know what to do with such return value anyway. But I prefer it this way, thanks for the approval.

you are right, as a follow up I'd update the code to


                    processFile(path == null || path.isEmpty() // Zero length string if the node is a root

-------------

PR: https://git.openjdk.org/jtharness/pull/35


More information about the jtharness-dev mailing list