Revision: 10800
Author:   [email protected]
Date:     Thu Feb 23 00:15:45 2012
Log:      Allow the ZWJ and ZWNJ characters in IdentifierPart as per ES5.

Patch from Mathias Bynens <[email protected]>
BUG=1958
TEST=Try `var a\u200c\u200d;` and confirm it doesn't throw an error.

Review URL: https://chromiumcodereview.appspot.com/9433031
Patch from Mathias Bynens <[email protected]>.
http://code.google.com/p/v8/source/detail?r=10800

Modified:
 /branches/bleeding_edge/AUTHORS
 /branches/bleeding_edge/src/char-predicates.h

=======================================
--- /branches/bleeding_edge/AUTHORS     Wed Feb 22 04:26:36 2012
+++ /branches/bleeding_edge/AUTHORS     Thu Feb 23 00:15:45 2012
@@ -31,6 +31,7 @@
 Jonathan Liu <[email protected]>
 Kun Zhang <[email protected]>
 Martyn Capewell <[email protected]>
+Mathias Bynens <[email protected]>
 Matt Hanselman <[email protected]>
 Maxim Mossienko <[email protected]>
 Michael Lutz <[email protected]>
=======================================
--- /branches/bleeding_edge/src/char-predicates.h       Tue May  3 01:23:58 2011
+++ /branches/bleeding_edge/src/char-predicates.h       Thu Feb 23 00:15:45 2012
@@ -57,6 +57,8 @@
   static inline bool Is(uc32 c) {
     return IdentifierStart::Is(c)
         || unibrow::Number::Is(c)
+        || c == 0x200C  // U+200C is Zero-Width Non-Joiner.
+        || c == 0x200D  // U+200D is Zero-Width Joiner.
         || unibrow::CombiningMark::Is(c)
         || unibrow::ConnectorPunctuation::Is(c);
   }

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to