On 12/03/2014 17:18, Mark Sheppard wrote:
This looks good. One other thing in this code (pre-dates your patch but your patch makes it a bit more obvious) is that if NewByteArray fails then we don't call FreeEnvironmentStringsA to free the environment.Hi an updated webrev as per suggested changes http://cr.openjdk.java.net/~msheppar/8036603/webrev.01/
-Alan