Purge Surrogate usages

Ulf Zibis Ulf.Zibis at gmx.de
Fri Mar 26 18:36:20 UTC 2010


Am 24.03.2010 00:50, schrieb Martin Buchholz:
> I've added another mini-patch to my patch set.
>
> http://cr.openjdk.java.net/~martin/webrevs/openjdk7/isBMPCodePoint3
>
> This deletes Surrogate.java, as Ulf wants,
> except that ... it's another variant of Surrogate.java!
> (which I didn't know existed)
>
> Uses of Surrogate.neededFor are all now changed to
> Character.isSupplementaryCodePoint, as suggested by Ulf.
>
> I intend to fold all of the isBMPCodePoint patches together into one
> before I commit them.
>
> Ulf, please review.
>    

Looking at my old patch:
https://bugs.openjdk.java.net/attachment.cgi?id=148&action=diff,
I'm afraid, that there are some remaining references to Surrogate class 
in the code base:
- cold imports
- static final constants
Can you declude them in your patch?

-Ulf





More information about the core-libs-dev mailing list