Thanks Roger and Alan!

I'll fix alignment before pushing.

Sincerely yours,
Ivan

On 11.08.2014 22:14, Alan Bateman wrote:
On 11/08/2014 18:14, Ivan Gerasimov wrote:
Hello!

There seems to be a small native memory leak in the ProcessBuilder's code.

Would you please help review the trivial fix for that?

BUGURL: https://bugs.openjdk.java.net/browse/JDK-8054841
WEBREV: http://cr.openjdk.java.net/~igerasim/8054841/0/webrev/
Looks okay to me too, you might want to just re-align the 3 parameter to releaseBytes as it looks like they were neatly aligned previously.

-Alan



Reply via email to