RFR 8189717: Too much documentation of ProcessBuilder.start copied to ProcessBuilder.startPipeline

Lance Andersen lance.andersen at oracle.com
Fri Jul 13 20:50:36 UTC 2018


Hi Roger,

+1

Have a great weekend!

Best
Lance
> On Jul 13, 2018, at 4:38 PM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
> 
> Hi Lance,
> 
> Updated in place with your suggestion.
> http://cr.openjdk.java.net/~rriggs/webrev-process-8189717/
> 
> Thanks, Roger
> 
> On 7/13/2018 4:31 PM, Lance Andersen wrote:
>> Hi Roger,
>> 
>>> On Jul 13, 2018, at 3:48 PM, Roger Riggs <Roger.Riggs at Oracle.com <mailto:Roger.Riggs at Oracle.com>> wrote:
>>> 
>>> Hi Lance,
>>> 
>>> How about:
>>> Subsequent modifications to any of {@code builder}swill not affect the returned {@link Process}.
>> ….. to any of the specified builders ….
>> 
>> At a minimum, you need  ‘the’ added to the above
>> 
>>> Roger
>>> 
>>> 
>>> On 7/13/2018 3:26 PM, Lance Andersen wrote:
>>>> Hi Roger,
>>>> 
>>>> I am struggling with the wording at:
>>>> 
>>>> ———
>>>> * Subsequent modifications to any of process builders in the {@code builders}
>>>> 1198 * argument will not affect the returned {@link Process}.
>>>> 
>>>> ————
>>>> 
>>>> Maybe specified/provided  process builders
>>>> 
>>>> Not sure, will think about it a bit.
>>>> 
>>>> Otherwise I think the changes are fine
>>>>> On Jul 13, 2018, at 3:07 PM, Roger Riggs <Roger.Riggs at oracle.com <mailto:Roger.Riggs at oracle.com>> wrote:
>>>>> 
>>>>> Please review a doc change to improve the description of ProcessBuilder.startPipeline.
>>>>> The clarifications are not intended to imply any change in behavior and only focus the description
>>>>> on the steps taken for each command in the pipeline.
>>>>> (A CSR will be needed also when the wording is finalized).
>>>>> 
>>>>> Webrev:
>>>>> http://cr.openjdk.java.net/~rriggs/webrev-process-8189717/ <http://cr.openjdk.java.net/%7Erriggs/webrev-process-8189717/>
>>>>> Issue:
>>>>> https://bugs.openjdk.java.net/browse/JDK-8189717
>>>>> 
>>>>> Thanks, Roger
>>>>> 
>>>> 
>>>> <oracle_sig_logo.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>>> Oracle Java Engineering
>>>> 1 Network Drive
>>>> Burlington, MA 01803
>>>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>>> 
>>>> 
>>>> 
>>> 
>> 
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering
>> 1 Network Drive
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>> 
>> 
>> 
> 

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list