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

Roger Riggs rriggs at openjdk.org
Wed Jul 2 14:37:47 UTC 2025


On Fri, 27 Jun 2025 16:23:59 GMT, Jaikiran Pai <jpai at openjdk.org> 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 lang = "java" :
>> 544:          *     Redirect.DISCARD.file() // is the filename appropriate for the operating system
>> 545:          *     Redirect.DISCARD.type() == Redirect.Type.WRITE
> 
> With this use of snippet here I think it no longer conveys what the original `{@code}` block was trying to convey. I can't currently think of a better proposal either.

Updated `Redirect.DISCARD.file() != null &&` // so its a valid expression.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25884#discussion_r2180227712


More information about the core-libs-dev mailing list