Re: RFR: 8266291: (jrtfs) Calling Files.exists may break the JRT filesystem [v2]

2021-05-14 Thread Athijegannathan Sundararajan
> Problematic paths that start with /modules/modules prefix are handled properly Athijegannathan Sundararajan has updated the pull request incrementally with one additional commit since the last revision: edited comment as per code review. - Changes: - all:

Re: RFR: 8266291: (jrtfs) Calling Files.exists may break the JRT filesystem

2021-05-14 Thread Alan Bateman
On Fri, 14 May 2021 05:39:10 GMT, Athijegannathan Sundararajan wrote: > Problematic paths that start with /modules/modules prefix are handled properly Marked as reviewed by alanb (Reviewer). src/java.base/share/classes/jdk/internal/jimage/ImageReader.java line 475: > 473: } >

Re: RFR: 8266291: (jrtfs) Calling Files.exists may break the JRT filesystem

2021-05-14 Thread Claes Redestad
On Fri, 14 May 2021 05:39:10 GMT, Athijegannathan Sundararajan wrote: > Problematic paths that start with /modules/modules prefix are handled properly LGTM - Marked as reviewed by redestad (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/4022

RFR: 8266291: (jrtfs) Calling Files.exists may break the JRT filesystem

2021-05-13 Thread Athijegannathan Sundararajan
Problematic paths that start with /modules/modules prefix are handled properly - Commit messages: - fixed typos in comments - 8266291: (jrtfs) Calling Files.exists may break the JRT filesystem Changes: https://git.openjdk.java.net/jdk/pull/4022/files Webrev: