[9] RFR 8130181: Deprecate java.security.Provider(String, double, String), add Provider(String, String, String)

[Wang Weijun]

In fact, now that getVersion() is deprecated, can we just always return 0d?

On the other hand, your parseVersionStr(s) is not able to handle "9.0d".

--Max

> On Jun 23, 2016, at 3:47 AM, Valerie Peng <valerie.peng at oracle.com> wrote:
> 
> Max,
> 
> Thanks for the comments~
> 
> The String.valueOf() calls are for handling null values.
> The name and id fields are also of type String and they use the String.valueOf() calls when putting in the Provider.id attribute values, so I follow the same convention. I also updated the parseVersionStr(String) utility method to handle null or invalid version strings though, and assign 0d if parsing fails.
> 
> According to the Verona JEP, "java.specification.version" should be $VNUM which is a non-empty sequence of non- negative integer numerals, without leading zeros, separated by period characters.
> I am not sure if we can assume the same format for supplied version string though. So, the current impl of parsing of version string can take the $VSTR (again in Verona JEP) which consists of $VUM optionally followed by pre-release and build info, i.e. $VNUM(-$PRE)?(\+$BUILD)?(-$OPT)?
> 
> Webrev updated:
> http://cr.openjdk.java.net/~valeriep/8130181/webrev.01
> 
> Valerie
> 
> On 6/15/2016 8:33 PM, Wang Weijun wrote:
>> You used String.valueOf(provider.getVersionStr()) on lines 70 and 861. Why not just provider.getVersionStr()?
>> 
>> For parseVersionStr(), where is the spec for "java.specification.version"? Can we just use regex /(^\d+(\.\d+)?)/ to match the substring and turn it into a double?
>> 
>> --Max