<AWT Dev> RFR [XS] : 8230900: missing ReleaseStringUTFChars in java.desktop native code

[Baesken, Matthias]

Hi Christoph,

I removed the  blanks after  ReleaseStringUTFChars  in fontpath.c,   and adjusted the  ReleaseStringUTFChars call to  C++ syntax in ShellFolder2.c.
New webrev :

http://cr.openjdk.java.net/~mbaesken/webrevs/8230900.1/

Best Regards ,
   Matthias


From: Langer, Christoph <christoph.langer at sap.com>
Sent: Donnerstag, 12. September 2019 21:29
To: Baesken, Matthias <matthias.baesken at sap.com>; 2d-dev at openjdk.java.net; awt-dev at openjdk.java.net
Subject: RE: RFR [XS] : 8230900: missing ReleaseStringUTFChars in java.desktop native code

Hi Matthias,

the changes seem to make sense.

I've got 2 findings:

In fontpath.c, there is a space between ReleaseStringUTFChars and ( which should be removed in lines 1282 and 1339.

In ShellFolder2.c, line 1134, the function call takes only 2 parameters. You need to remove the first param env. Otherwise it breaks the Windows build.

Thanks
Christoph

From: 2d-dev <2d-dev-bounces at openjdk.java.net<mailto:2d-dev-bounces at openjdk.java.net>> On Behalf Of Baesken, Matthias
Sent: Donnerstag, 12. September 2019 11:08
To: 2d-dev at openjdk.java.net<mailto:2d-dev at openjdk.java.net>; awt-dev at openjdk.java.net<mailto:awt-dev at openjdk.java.net>
Subject: [CAUTION] [OpenJDK 2D-Dev] RFR [XS] : 8230900: missing ReleaseStringUTFChars in java.desktop native code

Hello, please reviews this small fix .

It adds  a couple of missing ReleaseStringUTFChars  calls  in   java.desktop  module native  coding (mostly in cases of early returns).


Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8230900

http://cr.openjdk.java.net/~mbaesken/webrevs/8230900.0/

Thanks, Matthias
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/awt-dev/attachments/20190913/7fc89bde/attachment-0001.html>