RFR 9: 8071481: (Process) Merge UNIXProcess.java into ProcessImpl.java
chris.hegarty at oracle.com
Fri Jan 23 17:53:53 UTC 2015
On 23 Jan 2015, at 17:18, Peter Levart <peter.levart at 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.
> 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.
>> 8071481: (Process) Merge UNIXProcess.java into ProcessImpl.java
>>  https://bugs.openjdk.java.net/browse/JDK-8071481
More information about the core-libs-dev