On 2013/11/06 21:19:41, Lasse Reichstein Nielsen wrote:
drive-by comment :)
https://codereview.chromium.org/59853006/diff/50001/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/59853006/diff/50001/src/runtime.cc#newcode6209
src/runtime.cc:6209: // character is also ASCII. This is currently the
case,
but it
Is "ASCII" here still relevant?
In most cases, ASCII should be changed to OneByte after the change to
Latin1,
and we still have some work here. But in this case, it is actually still
correct: this fast pass only handles the ASCII range, converting between
a-z and
A-Z. Characters like Ä are not handled. In those cases, we simply drop to
the
slow case afterwards.
https://codereview.chromium.org/59853006/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
