RFR [9] 8041972: Add improved parse/format methods for Long/Integer (original) (raw)

Claes Redestad claes.redestad at oracle.com
Thu Jun 19 17:39:15 UTC 2014


Hi,

an updated webrev with reworked, public methods is available here: http://cr.openjdk.java.net/~redestad/8041972/webrev.8/

Reviews are yet again appreciated!

/Claes

On 06/17/2014 05:43 PM, Claes Redestad wrote:

Ok, I'm working on improving code and comments based on feedback. I'll split the SharedSecrets part out, make the methods public and post a new webrev soon.

/Claes On 06/17/2014 04:21 PM, roger riggs wrote: Yes, that looks more consistent with the current versions.

Though you want to see these for 8u, the preferred pattern is to make the changes in 9 and then backport the result (in this case adding the shared secrets aspect). Roger

On 6/16/2014 4:13 PM, Claes Redestad wrote: ... The terminology used in java.lang.String for offsets and indexes into strings would be provide a consistent base for talking about substrings. If we're taking cues from String.substring, I guess int beginIndex[, int fromIndex] would be more appropriate. How about: /** * Parses the character sequence argument in the specified {@code radix}, * beginning at the specified {@code beginIndex} and extending to the * character at index {@code endIndex - 1}. * * @see java.lang.Integer#parseInt(String, int) * @param s the {@code CharSequence} containing the integer * representation to be parsed * @param radix the radix to be used while parsing {@code s}. * @param beginIndex the beginning index, inclusive. * @param endIndex the ending index, exclusive. * @return the integer represented by the subsequence in the * specified radix. */ static int parseInt(CharSequence s, int radix, int beginIndex, int endIndex) ? Thanks! /Claes



More information about the core-libs-dev mailing list