Thanks!
http://codereview.chromium.org/6072004/diff/1/src/heap-inl.h File src/heap-inl.h (right): http://codereview.chromium.org/6072004/diff/1/src/heap-inl.h#newcode45 src/heap-inl.h:45: // Check for ascii first since this is the common case. On 2010/12/21 12:48:12, Lasse Reichstein wrote:
ASCII is an acronym, even though we might not respect that in function
names.
More instances below.
Done. http://codereview.chromium.org/6072004/diff/1/src/heap-inl.h#newcode47 src/heap-inl.h:47: if (static_cast<uc16>(str[i]) > String::kMaxAsciiCharCode) { On 2010/12/21 12:48:12, Lasse Reichstein wrote:
Just cast to uint8_t, and compare to String:kMaxAsciiCharCodeU. It
will allow
the compiler to just check the sign bit.
Done. http://codereview.chromium.org/6072004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
