String.lastIndexOf confused by unpaired trailing surrogate

[Martin Buchholz]

Ulf, Sherman, Masayoshi,
here are changes for you to review.
Only the patch highSurrogate needs a separate bug filed
(and CCC, please)

Ulf, I've made some progress on integrating your changes,
although almost all of them have been somewhat martinized:

Ulf-style tidying, mostly whitespace.
[mq]: Character-warnings2
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/Character-warnings2

Very minor optimizations.  Barely worth doing.
Note my removal of the need to have n++ inside the loop.
imported patch ulf-opto
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/ulf-opto

Addition of highSurrogate and lowSurrogate
imported patch highSurrogate
http://cr.openjdk.java.net/~martin/webrevs/openjdk7/highSurrogate

Martin

On Sat, Mar 20, 2010 at 17:24, Ulf Zibis <Ulf.Zibis at gmx.de> wrote:
> Sherman, please again consider about shifting Surrogate.high/low to
> Character.high/lowSurrogate.