review request for 6798511/6860431: Include functionality of Surrogate in Character
Martin Buchholz
martinrb at google.com
Sun Mar 21 07:20:20 UTC 2010
On Sat, Mar 20, 2010 at 17:13, Ulf Zibis <Ulf.Zibis at gmx.de> wrote:
> Am 20.03.2010 01:13, schrieb Martin Buchholz:
> Don't you think we should add a hint to javadoc to inform the user about the
> implementation difference between isSupplementaryCodePoint and
> isValidCodePoint?
No.
> It's likely, the user would use isBMPCodePoint and isSupplementaryCodePoint
> as pair, not knowing about the performance problem.
I don't think it's a performance problem in the real world.
We don't usually put such performance information in the javadoc.
Can you demonstrate a performance advantage
of your implementation of isSupplementaryCodePoint
for BMP characters, when there is no call to
isBMPCodePoint? (Such a demonstration typically
requires testing on a large variety of systems and JITs)
Martin
More information about the core-libs-dev
mailing list