String.lastIndexOf confused by unpaired trailing surrogate

[Martin Buchholz]

Masayoshi,

Ulf and I are working on a few changes to supplementary character handling.
Character.isSurrogate has already gone in.

The following are in the pipeline:

6934268: Better implementation of Character.isValidCodePoint
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/isValidCodePoint
6934265: Add public method Character.isBMPCodePoint
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/public-isBMPCodePoint
[mq]: isBMPCodePoint2
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/isBMPCodePoint2
6937112: String.lastIndexOf confused by unpaired trailing surrogate
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/lastIndexOf

In addition, Ulf and I would like to add
char Character.highSurrogate(int codePoint)
char Character.lowSurrogate(int codePoint)

Ulf,
please provide me with your latest patch for Character.highSurrogate
and I will add it to the pipeline.

Martin

On Tue, Mar 23, 2010 at 13:37, Xueming Shen <Xueming.Shen at sun.com> wrote:
> CCed Masayoshi.
>
> Masayoshi, Martin and Ulf are doing some "small" overhaul on those
> supplementary methods guess
> you might be interested to review the change.
>
> Martin, Ulf, please CC Masayoshi if you are touching the supplementary
> handling code.
>