+1 You'll have to update the copyright years in the jni_util files.
Cheers Christoph > -----Original Message----- > From: David Holmes <[email protected]> > Sent: Mittwoch, 30. Oktober 2019 05:29 > To: Alexey Ivanov <[email protected]>; core-libs <core-libs- > [email protected]>; hotspot-runtime <hotspot-runtime- > [email protected]>; Langer, Christoph <[email protected]> > Cc: Claes Redestad <[email protected]> > Subject: Re: RFR JDK-8232724: Remove indirection with calling > JNU_NewStringPlatform > > LGTM. > > Thanks, > David > > On 30/10/2019 1:04 am, Alexey Ivanov wrote: > > Hi David, Christoph, > > > > Thank you for your reviews. > > > > On 29/10/2019 07:49, David Holmes wrote: > >>> Shall I remove one of them? The one in jvm.h because it's unused? > >> > >> Yes please remove the unused one in jvm.h. > > > > The updated webrev: > > http://cr.openjdk.java.net/~aivanov/8232724/webrev.01/ > > > > -- > > Regards, > > Alexey
