Reviewers: Michael Starzinger,
Message:
This relands the CL that was rolled back with r19199. Major changes (in
second
patch set):
- Removed test/webkit/string-trim.js as suggested by machenbach@. Adapted
duplicate exists in test/mjsunit/third_party/
- v8::internal::Whitespace are now characters that are listed in ECMA-262
5.1,
7.2, and Unicode characters of category 'Zs' (unibrow::WhiteSpace). It no
longer
includes LineTerminator characters and NEL. (The old notion of WhiteSpace
in V8
actually meant both WhiteSpace *and* LineTerminator values)
- Removed unused table for "Space" in unicode.*.
- Introduced new char-predicate WhiteSpaceOrLineTerminator that includes
both
WhiteSpace and LineTerminator values as per spec, 7.2 and 7.3.
Description:
Fix inconsistencies wrt whitespaces.
This relands r19196 with fixes.
BUG=v8:3109
LOG=Y
Please review this at https://codereview.chromium.org/141323007/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+206, -305 lines):
D test/webkit/string-trim-expected.txt
D test/webkit/string-trim.js
M src/char-predicates.h
M src/conversions-inl.h
M src/dateparser.h
M src/jsregexp.cc
M src/runtime.cc
M src/scanner.h
M src/scanner.cc
M src/unicode.h
M src/unicode.cc
M test/cctest/test-regexp.cc
M test/mjsunit/third_party/string-trim.js
A test/mjsunit/whitespaces.js
--
--
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.
