Reviewed. -Chris
> On 22 Jun 2014, at 19:18, Seán Coffey <sean.cof...@oracle.com> wrote: > > The JDK-8036603 fix pushed on Friday broke windows builds. Bad macro call. > Simple fix : > https://bugs.openjdk.java.net/browse/JDK-8047777 > applicable to jdk8u-dev only. > >> --- a/src/windows/native/java/lang/ProcessEnvironment_md.c >> +++ b/src/windows/native/java/lang/ProcessEnvironment_md.c >> @@ -41,7 +41,7 @@ >> CHECK_NULL_RETURN(string_class, NULL); >> String_init_ID = >> (*env)->GetMethodID(env, string_class, "<init>", "([B)V"); >> - CHECK_NULL_RETURN(String_init_ID); >> + CHECK_NULL_RETURN(String_init_ID, NULL); >> blockA = (jbyte *) GetEnvironmentStringsA(); >> if (blockA == NULL) { >> /* Both GetEnvironmentStringsW and GetEnvironmentStringsA > > regards, > Sean.
