[7u-dev] Request for review of backport 7147084
Alan Bateman
Alan.Bateman at oracle.com
Wed Oct 9 10:06:08 UTC 2013
On 09/10/2013 08:47, Ivan Gerasimov wrote:
> Alan, Rob, Martin,
>
> Would you please help review the backport? I see that you were the
> reviewers for the master fix.
> I need a review for the backport, since the patch couldn't get applied
> cleanly.
>
> Here are the changes I had to make:
> - I had to manually replace Java_java_lang_ProcessImpl_create()
> function body with the new version, as 'hg patch' could not handle it
> automatically.
> - I had to replace wide-char strings to regular char*, in all the
> calls to win32Error() function.
>
> Thanks in advance,
> Ivan
>
I looked over the webrev and it seems to match the jdk8 changes except
for the win32Error usage. So I think it's okay. One thing to mention is
that there is a patch under review to fix issues with the test so this
should probably be pushed to jdk7u-dev too.
-Alan.
More information about the core-libs-dev
mailing list