codereview request: 6639443/ (original) (raw)

Martin Buchholz martinrb at google.com
Tue Jul 14 20:15:15 UTC 2009


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>



More information about the core-libs-dev mailing list