[OpenJDK 2D-Dev] [9] RFR: JDK-8151110, , libfontmanager should free memory with delete[] if it was allocated with new[]

2016-03-15 Thread prasanta sadhukhan
Hi Phil, Please review a small fix for bug: https://bugs.openjdk.java.net/browse/JDK-8151110 webrev: http://cr.openjdk.java.net/~psadhukhan/8151110/webrev.00/ where the issue was found that the allocation of memory was done through new operator but free was done through free(). Since they opera

Re: [OpenJDK 2D-Dev] [9] RFR: JDK-8151110, , libfontmanager should free memory with delete[] if it was allocated with new[]

2016-03-19 Thread Sergey Bylokhov
Looks fine. On 16.03.16 21:46, Philip Race wrote: +1 -phil. On 3/15/16, 11:33 PM, prasanta sadhukhan wrote: Hi Phil, Please review a small fix for bug: https://bugs.openjdk.java.net/browse/JDK-8151110 webrev: http://cr.openjdk.java.net/~psadhukhan/8151110/webrev.00/ where the issue was foun

Re: [OpenJDK 2D-Dev] [9] RFR: JDK-8151110, , libfontmanager should free memory with delete[] if it was allocated with new[]

2016-03-19 Thread Philip Race
+1 -phil. On 3/15/16, 11:33 PM, prasanta sadhukhan wrote: Hi Phil, Please review a small fix for bug: https://bugs.openjdk.java.net/browse/JDK-8151110 webrev: http://cr.openjdk.java.net/~psadhukhan/8151110/webrev.00/ where the issue was found that the allocation of memory was done through ne