| Index: src/regexp/regexp-parser.cc
|
| diff --git a/src/regexp/regexp-parser.cc b/src/regexp/regexp-parser.cc
|
| index 68c5a6d2633cd31ee8ee17c0487ec87ea44d7b7d..c567759ab4b1973e24fababddaa64aa6f88e1187 100644
|
| --- a/src/regexp/regexp-parser.cc
|
| +++ b/src/regexp/regexp-parser.cc
|
| @@ -703,10 +703,6 @@ void RegExpParser::ScanForCaptures() {
|
| if (current() != '<') break;
|
|
|
| if (FLAG_harmony_regexp_lookbehind) {
|
| - // TODO(jgruber): To be more future-proof we could test for
|
| - // IdentifierStart here once it becomes clear whether group names
|
| - // allow unicode escapes.
|
| - // https://github.com/tc39/proposal-regexp-named-groups/issues/23
|
| Advance();
|
| if (current() == '=' || current() == '!') break;
|
| }
|
| @@ -780,8 +776,6 @@ const ZoneVector<uc16>* RegExpParser::ParseCaptureGroupName() {
|
|
|
| // Convert unicode escapes.
|
| if (c == '\\' && current() == 'u') {
|
| - // TODO(jgruber): Reconsider this once the spec has settled.
|
| - // https://github.com/tc39/proposal-regexp-named-groups/issues/23
|
| Advance();
|
| if (!ParseUnicodeEscape(&c)) {
|
| ReportError(CStrVector("Invalid Unicode escape sequence"));
|
|
|