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.

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:

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

2016-07-28 Thread Paul Sandoz
> 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 change

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

2016-07-27 Thread Claes Redestad
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 change looks good. We provide a shortcut for the common case where only the major version number is of