Re: [jdk25] RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-07-07 Thread Iris Clark
On Mon, 7 Jul 2025 18:29:34 GMT, Roger Riggs wrote: > Identical doc-only backport to jdk 25. Already approved in the CSR. Changes appear to be identical to those in `master` branch. - Marked as reviewed by iris (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/26168#pullrequ

[jdk25] RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-07-07 Thread Roger Riggs
Identical doc-only backport to jdk 25. Already approved in the CSR. - Commit messages: - Backport afb4a1be9e5dc2a9c0d812f5a36717c9f82241a9 Changes: https://git.openjdk.org/jdk/pull/26168/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26168&range=00 Issue: https://bugs.op

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v4]

2025-07-03 Thread Jaikiran Pai
On Thu, 3 Jul 2025 14:23:00 GMT, Roger Riggs wrote: >> src/java.base/share/classes/java/lang/Process.java line 113: >> >>> 111: * releasing the operating system resources. >>> 112: * {@code Try-with-resources} can be used to open and close the >>> streams. >>> 113: * For example, to capture

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v4]

2025-07-03 Thread Roger Riggs
On Thu, 3 Jul 2025 11:46:32 GMT, Jaikiran Pai wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix links per review comments and snippet syntax in >> ProcessBuilder.Redirect.DISCARD. > > src/java.base/share/classes/

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v4]

2025-07-03 Thread Jaikiran Pai
On Wed, 2 Jul 2025 14:52:06 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Roger Riggs
On Wed, 2 Jul 2025 13:02:04 GMT, Jaikiran Pai wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> javadoc cleanup > > src/java.base/share/classes/java/lang/Process.java line 208: > >> 206: * >> 207: * @apiNot

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Jaikiran Pai
On Wed, 2 Jul 2025 14:46:20 GMT, Roger Riggs wrote: > But including both links makes the sentence hard to read, so I picked the > more expressive reader method to link to. It didn't realize you had intentionally used the `CharSet` one. If you think the CharSet would provide better guidance to

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v4]

2025-07-02 Thread Jaikiran Pai
On Wed, 2 Jul 2025 14:49:01 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v4]

2025-07-02 Thread Roger Riggs
> Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add advice and example using try-with-resources to open an

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Jaikiran Pai
On Tue, 1 Jul 2025 22:30:57 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Roger Riggs
On Fri, 27 Jun 2025 16:23:59 GMT, Jaikiran Pai wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> javadoc cleanup > > src/java.base/share/classes/java/lang/ProcessBuilder.java line 545: > >> 543: * {@snippet

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Jaikiran Pai
On Tue, 1 Jul 2025 22:30:57 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Jaikiran Pai
On Tue, 1 Jul 2025 22:30:57 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-02 Thread Jaikiran Pai
On Tue, 1 Jul 2025 22:30:57 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader. >

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v3]

2025-07-01 Thread Roger Riggs
> Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add advice and example using try-with-resources to open an

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v2]

2025-07-01 Thread Roger Riggs
On Fri, 27 Jun 2025 19:06:22 GMT, Roger Riggs wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader.

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup [v2]

2025-06-27 Thread Roger Riggs
> Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add advice and example using try-with-resources to open an

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Roger Riggs
On Fri, 27 Jun 2025 16:02:45 GMT, Jaikiran Pai wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader.

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Wed, 18 Jun 2025 20:16:18 GMT, Roger Riggs wrote: > Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Wed, 18 Jun 2025 20:16:18 GMT, Roger Riggs wrote: > Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Fri, 27 Jun 2025 16:12:53 GMT, Jaikiran Pai wrote: >> src/java.base/share/classes/java/lang/Process.java line 202: >> >>> 200: * >>> 201: * @apiNote >>> 202: * Avoid using both {@link #getInputStream} and {@link >>> #inputReader(Charset)}. >> >> The reworded sentence I think

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Fri, 27 Jun 2025 16:09:02 GMT, Jaikiran Pai wrote: >> Improve the documentation of Process use of system resources. >> >> Describe the implementation closing streams when no longer referenced. >> Clarify the interactions between inputStream and inputReader and errorStream >> and errorReader.

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Wed, 18 Jun 2025 20:16:18 GMT, Roger Riggs wrote: > Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-27 Thread Jaikiran Pai
On Wed, 18 Jun 2025 20:16:18 GMT, Roger Riggs wrote: > Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add

Re: RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-23 Thread Volkan Yazici
On Wed, 18 Jun 2025 20:16:18 GMT, Roger Riggs wrote: > Improve the documentation of Process use of system resources. > > Describe the implementation closing streams when no longer referenced. > Clarify the interactions between inputStream and inputReader and errorStream > and errorReader. > Add

RFR: 8354872: Clarify java.lang.Process resource cleanup

2025-06-18 Thread Roger Riggs
Improve the documentation of Process use of system resources. Describe the implementation closing streams when no longer referenced. Clarify the interactions between inputStream and inputReader and errorStream and errorReader. Add advice and example using try-with-resources to open and close stre