Re: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception

2018-10-31 Thread Sergey Bylokhov
+1 On 30/10/2018 07:05, Thomas Stüfe wrote: Seems fine and trivial. Regards, Thomas On Tue, Oct 30, 2018 at 2:56 PM Volker Simonis wrote: Hi, can I please have a review for the following tiny change which fixes the license header on a few AIX-specific files:

Re: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception

2018-10-31 Thread Volker Simonis
Thanks for the quick reviews! Volker On Tue, Oct 30, 2018 at 4:35 PM Roger Riggs wrote: > > Looks fine, Reviewed. > > Thanks, Roger > > On 10/30/2018 11:02 AM, Baesken, Matthias wrote: > > Hi Volker, looks good (not a Reviewer however) ! > > Maybe you should also adjust the Copyright year

Re: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception

2018-10-30 Thread Roger Riggs
Looks fine,  Reviewed. Thanks, Roger On 10/30/2018 11:02 AM, Baesken, Matthias wrote: Hi Volker, looks good (not a Reviewer however) ! Maybe you should also adjust the Copyright year info in the files you touched . Best regards, Matthias -Original Message- Hi, can I please have

RE: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception

2018-10-30 Thread Baesken, Matthias
Hi Volker, looks good (not a Reviewer however) ! Maybe you should also adjust the Copyright year info in the files you touched . Best regards, Matthias -Original Message- Hi, can I please have a review for the following tiny change which fixes the license header on a few

Re: RFR(XS): 8213151: [AIX] Some class library files are missing the Classpath exception

2018-10-30 Thread Thomas Stüfe
Seems fine and trivial. Regards, Thomas On Tue, Oct 30, 2018 at 2:56 PM Volker Simonis wrote: > > Hi, > > can I please have a review for the following tiny change which fixes > the license header on a few AIX-specific files: > > http://cr.openjdk.java.net/~simonis/webrevs/2018/8213151/ >