On 12/03/2014 17:18, Mark Sheppard wrote:
Hi
   an updated webrev as per suggested changes
http://cr.openjdk.java.net/~msheppar/8036603/webrev.01/
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.

-Alan

Reply via email to