Hi Xuelei,

thanks for looking.

The warnings would appear in my Eclipse IDE. During the tests (build) nothing 
had been observed and this will hopefully still be the same. 😊

So you are good with the changes?

Best regards
Christoph

> -----Original Message-----
> From: Xuelei Fan <xuelei....@oracle.com>
> Sent: Mittwoch, 8. Mai 2019 15:58
> To: Langer, Christoph <christoph.lan...@sap.com>; security-dev <security-
> d...@openjdk.java.net>
> Subject: Re: RFR(XS): 8223555: Cleanups in cacerts tests
> 
> Hi Christoph,
> 
> Did you see compiler warning for the files?  I tried with the JDK
> repository build, no compiler warning message.  The update is safe.  I
> may miss something about why we want to make the update.
> 
> Thanks,
> Xuelei
> 
> On 5/8/2019 1:21 AM, Langer, Christoph wrote:
> > Hi,
> >
> > please review this small change which contains a few minor cleanups to
> > the tests for the CA certificates. I had it in my mercurial queue for
> > quite some time and always wanted to contribute it 😊
> >
> > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8223555.0/
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8223555
> >
> > Thanks
> >
> > Christoph
> >

Reply via email to