Skip to content

Make private field name parsing ecma 262 compliant #50883

Open
@amcasey

Description

@amcasey

This file has parse errors, but probably shouldn't.

Considerations:

  • Improve escape sequence handling in private names #50856 added support for starting with a unicode escape
  • The names #ZW__NJ and #ZW__J contain \u200c and \u200d, respectively (not here, but in the file)
  • Support for extended unicode escapes (i.e. \u{6F}, rather than \u006F) depends on ES2015, but we appear to parse the file as ES5.
  • Converting the extended unicode escapes to regular unicode escapes does not seem to eliminate the errors

Discovered as part of #50835

Metadata

Metadata

Labels

BugA bug in TypeScriptEffort: ModerateRequires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".Help WantedYou can do thisRescheduledThis issue was previously scheduled to an earlier milestone

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions