Hi Chris,

I considered updating BigFork but would have had to update the stacktrace lines numbers, etc.
And I think it will be recognized that it is a dated reference.

Roger

On 1/23/2015 2:03 PM, Chris Hegarty wrote:
On 23 Jan 2015, at 18:57, Roger Riggs <roger.ri...@oracle.com> wrote:

Hi Peter, Chris,

Thanks for the reviews.

I found one more reference to UNIXProcess in a comment in childproc.h
Ah, good find.

Another reference in comments in BigFork.java, but probably doesn’t make sense 
to change at this point.

-Chris.

and will wait for a jprt run to finish.

Webrev:

http://cr.openjdk.java.net/~rriggs/webrev-unixprocess-rename-8071481/

Thanks, Roger

On 1/23/2015 12:53 PM, Chris Hegarty wrote:
On 23 Jan 2015, at 17:18, Peter Levart <peter.lev...@gmail.com> wrote:

Hi Roger,

Nice move/rename/merge.

I compared old UNIXProcess.java with new ProcessImpl.java using KDiff3 and they 
match perfectly as well as including the old ProcessImpl.java code.
Nice. I did the same just to see it for myself.  The other parts of the webrev 
look good too.

-Chris.

Regards, Peter

On 01/23/2015 05:11 PM, Roger Riggs wrote:
Please review this merge of UNIXProcess.java with ProcessImpl.java.

For Unix variants of Process, the implementation is split between
src/java.base/unix/classes/java/lang/ProcessImpl.java and UNIXProcess.java.
ProcessImpl.java contains only static methods and delegates to UNIXProcess.java.
Merging UNIXProcess.java into ProcessImpl.java removes a redundant method
and aligns the source files with the Windows implementation that contains only 
ProcessImpl.java.

Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-unixprocess-rename-8071481/

Issue:
  8071481: (Process) Merge UNIXProcess.java into ProcessImpl.java

[1] https://bugs.openjdk.java.net/browse/JDK-8071481

Reply via email to