review request for 6798511/6860431: Include functionality of Surrogate in Character

Ulf Zibis Ulf.Zibis at gmx.de
Sun Mar 21 00:13:47 UTC 2010


Am 20.03.2010 01:13, schrieb Martin Buchholz:
> We can do it for free
> by switching  =>  isValidCodePoint,
> so why not?
>    

Don't you think we should add a hint to javadoc to inform the user about 
the implementation difference between isSupplementaryCodePoint and 
isValidCodePoint?

It's likely, the user would use isBMPCodePoint and 
isSupplementaryCodePoint as pair, not knowing about the performance problem.

But more I would like to see 6932837 && 6935994 gets fixed, so we could 
stay on conservative style.

-Ulf








More information about the core-libs-dev mailing list