review request 6933322 - Add methods highSurrogate(), lowSurrogate() to class Character
martinrb at google.com
Fri Apr 2 13:01:18 PDT 2010
Writing good spec is hard and annoying, but important.
I've improved the spec in various ways, mostly as you suggested.
Please see the updated webrev
On Thu, Apr 1, 2010 at 01:42, Masayoshi Okutsu <Masayoshi.Okutsu at sun.com> wrote:
> Hi Martin,
> Here are my comments on the 6933322 changes.
> - I'd suggest that the Unicode terms be used instead of "the first part" and
> "the second part", something like "the high surrogate (also known as leading
> surrogate) code unit of the surrogate pair." If you want to emphasize the
> order, "the leading surrogate (high surrogate) code unit" should be OK.
> Actually there were some discussions about high/low vs. leading/trailing in
> JSR 204, and we decided to use high/low to follow the (main) Unicode terms.
> - @param, @return and @since are missing. There should be @see for the
> Otherwise, the changes look good to me, assuming that CCC would approve the
> API change.
> On 3/26/2010 5:52 AM, Ulf Zibis wrote:
>> Updated topic.
>> Am 25.03.2010 21:42, schrieb Ulf Zibis:
>>> Am 24.03.2010 09:24, schrieb Martin Buchholz:
>>>> Ulf, Sherman, Masayoshi,
>>>> here are changes for you to review.
>>>> Only the patch highSurrogate needs a separate bug filed
>>>> (and CCC, please)
>>> I had just filed it 2 weeks ago, see:
More information about the core-libs-dev