RE: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-08-01 Thread Iris Clark
Hi, Claes. > http://cr.openjdk.java.net/~redestad/8162439/webrev.02/ This looks good. Thanks, Iris

Re: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-29 Thread Paul Sandoz
> On 29 Jul 2016, at 05:15, Claes Redestad wrote: > > Hi Paul, > > On 07/28/2016 02:55 PM, Paul Sandoz wrote: >>> On 27 Jul 2016, at 19:36, Claes Redestad wrote: >>> >>> On 07/25/2016 08:01 PM, Iris Clark wrote: Hi, Claes. > Webrev: http://cr.openjdk.java.net/~redestad/8162439/

Re: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-28 Thread Claes Redestad
Hi Paul, On 07/28/2016 02:55 PM, Paul Sandoz wrote: On 27 Jul 2016, at 19:36, Claes Redestad wrote: On 07/25/2016 08:01 PM, Iris Clark wrote: Hi, Claes. Webrev: http://cr.openjdk.java.net/~redestad/8162439/webrev.01/ Bug: https://bugs.openjdk.java.net/browse/JDK-8162439 I think that this c

Re: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-28 Thread Paul Sandoz
> /Claes > >> >> Regards, >> iris >> (not a Reviewer) >> >> -----Original Message- >> From: Claes Redestad >> Sent: Saturday, July 23, 2016 9:24 AM >> To: core-libs-dev@openjdk.java.net core-libs-dev >> Subject: RFR: 8162

Re: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-27 Thread Claes Redestad
d Sent: Saturday, July 23, 2016 9:24 AM To: core-libs-dev@openjdk.java.net core-libs-dev Subject: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions Hi, please review this patch to address a startup regression due to use of Runtime.Version.parse("8") etc in JarFile,

RE: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-25 Thread Iris Clark
ified. Regards, iris (not a Reviewer) -Original Message- From: Claes Redestad Sent: Saturday, July 23, 2016 9:24 AM To: core-libs-dev@openjdk.java.net core-libs-dev Subject: RFR: 8162439: Runtime.Version.parse needs fast-path for major versions Hi, please review this patch to address a s

RFR: 8162439: Runtime.Version.parse needs fast-path for major versions

2016-07-23 Thread Claes Redestad
Hi, please review this patch to address a startup regression due to use of Runtime.Version.parse("8") etc in JarFile, as introduced by JDK-8150680. This solution introduce a fast-path in case of what appears to be a single number is sent to Runtime.Version.parse to avoid initializing Runtime.