RFR: 8290359: Ensure that all directory streams are closed in jdk.link

2022-07-15 Thread Ryan Ernst
This commit adds try-with-resources for uses of Stream from Files methods that walk directories. - Commit messages: - 8290359: Ensure that all directory streams are closed in jdk.link Changes: https://git.openjdk.org/jdk/pull/9520/files Webrev: https://webrevs.openjdk.org/?repo=jdk

Re: RFR: 8290359: Ensure that all directory streams are closed in jdk.link

2022-07-18 Thread Chris Hegarty
On Fri, 15 Jul 2022 16:18:17 GMT, Ryan Ernst wrote: > This commit adds try-with-resources for uses of Stream from Files > methods that walk directories. Changes requested by chegar (Reviewer). src/jdk.jlink/share/classes/jdk/tools/jlink/internal/JlinkTask.java line 834: > 832:

Re: RFR: 8290359: Ensure that all directory streams are closed in jdk.link [v2]

2022-07-18 Thread Ryan Ernst
> This commit adds try-with-resources for uses of Stream from Files > methods that walk directories. Ryan Ernst has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request c

Re: RFR: 8290359: Ensure that all directory streams are closed in jdk.link [v2]

2022-07-18 Thread Ryan Ernst
On Mon, 18 Jul 2022 15:35:25 GMT, Chris Hegarty wrote: >> Ryan Ernst has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains three additional >> commits

Re: RFR: 8290359: Ensure that all directory streams are closed in jdk.link [v2]

2022-07-19 Thread Chris Hegarty
On Mon, 18 Jul 2022 18:22:02 GMT, Ryan Ernst wrote: >> This commit adds try-with-resources for uses of Stream from Files >> methods that walk directories. > > Ryan Ernst has updated the pull request with a new target base due to a merge > or a rebase. The incremental webrev excludes the unrelate