codereview request: 6639443/ (original) (raw)
Martin Buchholz martinrb at google.com
Tue Jul 14 20:15:15 UTC 2009
- Previous message: codereview request: 6639443/
- Next message: codereview request: 6639443/
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Tue, Jul 14, 2009 at 12:50, Xueming Shen <Xueming.Shen at sun.com> wrote:
(2) a) return (int)(char) uc ==uc; is nice:-) but I would go with the "more easy to read" return uc < Surrogate.UCS4MIN; if it were my code. Is there a big performance gain by doing that?
That's almost, but not exactly the same - uc might be negative. (I don't know whether that can ever happen, though)
My code is likely to be slightly faster than
return uc < Surrogate.UCS4_MIN && uc >= 0;
b) The "buffer" version and the "array" version of generate() are not synced.
Yikes! Good catch. Fixed - webrev regenerated.
Thanks,
Martin -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20090714/2cd59bd0/attachment.html>
- Previous message: codereview request: 6639443/
- Next message: codereview request: 6639443/
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]