Looks good to me -Pankaj
-----Original Message----- From: Sergey Bylokhov Sent: Thursday, March 5, 2020 2:35 PM To: Toshio 5 Nakamura <toshi...@jp.ibm.com>; awt-...@openjdk.java.net; 2d-dev <2d-dev@openjdk.java.net> Subject: Re: <AWT Dev> RFR: 8240518: Incorrect JNU_ReleaseStringPlatformChars in Windows Print (CC) 2d-dev, Looks fine. On 3/4/20 4:48 am, Toshio 5 Nakamura wrote: > Hi, > > I'd like to ask review and sponsor of this fix. > > Issue: https://bugs.openjdk.java.net/browse/JDK-8240518 > Fix: http://cr.openjdk.java.net/~tnakamura/8240518/webrev.00 > > Eclipse OpenJ9 VM detects two errors about JNU_ReleaseStringPlatformChars in > WPrinterJob.cpp. > Then, I checked similar situation in the folder, and found one lacking of the > release method in awt_PrintControl.cpp. > > Best regards, > > Toshio Nakamura > -- Best regards, Sergey.