[Webkit-unassigned] [Bug 78908] `\u200c` and `\u200d` should be allowed in IdentifierStart, as per ES5

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=78908





--- Comment #11 from Gavin Barraclough <barraclough at apple.com>  2012-02-22 14:30:20 PST ---
Hi Mathias,

The patch looks great - thanks for fixing this!

However we do ask for bug fixes to be accompanied by regression tests.  Do you think it would be possible to add one?

You should be able to test this with something like:

shouldBe("var x\u200c = 42; x\u200c", "42");
shouldBe("var x\u200d = 43; x\u200d", "43");

You'll find the tests in LayoutTests/fast/js, just pick an existing test to add the tests to, or if none look appropriate create a new .html / .js based on an existing one.  Let me know if you have an questions.

Many thanks,
G.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.