RFR: 8273961: jdk/nio/zipfs/ZipFSTester.java fails if file path contains '+' character [v2]

Remilia Scarlet github.com+22913521+1996scarlet at openjdk.java.net
Tue Sep 21 10:17:36 UTC 2021


On Tue, 21 Sep 2021 08:19:00 GMT, Christoph Langer <clanger at openjdk.org> wrote:

>> Remilia Scarlet has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR.
>
> test/jdk/jdk/nio/zipfs/ZipFSTester.java line 704:
> 
>> 702:         // character
>> 703:         return FileSystems.newFileSystem(
>> 704:             new URI("jar", URLDecoder.decode(encoded, "utf8"),
> 
> instead of using local variable encoded, I would just add the call to .replace("+", "%2b") inline.
> 
> The comments look good, I would suggest to put them between line 702/703.

Done. A new patch has been uploaded.

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

PR: https://git.openjdk.java.net/jdk/pull/5574


More information about the nio-dev mailing list