Hi Mark,

Looks fine as is.

The malloc could also be moved inside the if (SetupI18nProps...
but that's a bigger change.

Roger


On 2/18/2015 9:44 AM, Mark Sheppard wrote:
Hi
   please oblige and review the small change
http://cr.openjdk.java.net/~msheppar/8055204/webrev/

which addresses the parfait issue raised in
https://bugs.openjdk.java.net/browse/JDK-8055204

frees previously allocated memory in an error return path.

regards
Mark

Reply via email to