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