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

Claes Redestad claes.redestad at oracle.com
Tue Jun 17 15:43:32 UTC 2014

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