Hi Pavel,
These changes look fine.
I think you can update the JEP to match the updated patterns mentioned
below.
Regards, Roger
On 4/3/2017 7:50 PM, Paul Sandoz wrote:
On 3 Apr 2017, at 16:40, Pavel Rappo wrote:
On 3 Apr 2017, at 20:39, Paul Sandoz wrote:
Hi,
962 * [1-9][0-9]*((\.0)
> On 3 Apr 2017, at 16:40, Pavel Rappo wrote:
>
>
>> On 3 Apr 2017, at 20:39, Paul Sandoz wrote:
>>
>> Hi,
>>
>> 962 * [1-9][0-9]*((\.0)*\.[1-9][0-9]*)*
>>
>> You removed the initial “^” which is still mentioned in JDK-8148822
>>
>> < `^[1-9][0-9]*(((\.0)*\.[1-9][0-9]*)*)*$`. The sequence
> On 3 Apr 2017, at 20:39, Paul Sandoz wrote:
>
> Hi,
>
> 962 * [1-9][0-9]*((\.0)*\.[1-9][0-9]*)*
>
> You removed the initial “^” which is still mentioned in JDK-8148822
>
> < `^[1-9][0-9]*(((\.0)*\.[1-9][0-9]*)*)*$`. The sequence may be of arbitrary
> ---
>> `^[1-9][0-9]*((\.0)*\.[1-9][0-9]*
Hi,
962 * [1-9][0-9]*((\.0)*\.[1-9][0-9]*)*
You removed the initial “^” which is still mentioned in JDK-8148822
< `^[1-9][0-9]*(((\.0)*\.[1-9][0-9]*)*)*$`. The sequence may be of arbitrary
---
> `^[1-9][0-9]*((\.0)*\.[1-9][0-9]*)*$`. The sequence may be of arbitrary
JEP 223 is still using the f
Hello,
Please review the following change:
http://cr.openjdk.java.net/~prappo/8177738/webrev.00/
This is an attempt to resolve a group of issues related to Version-String API
and its implementation:
8177738: Runtime.Version must be a value-based class
8148822: (spec) Regex in Runtim
