chakra-core/ChakraCore

[RegExp] Exceptions for RegExp in Annex B of the ES2017 spec shouldn't be applied when the "unicode" flag is present

Open

#518 geöffnet am 10. März 2016

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.374 Forks)batch import
BugSeverity: 2help wanted

Repository-Metriken

Stars
 (9.000 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Annex B.1.4 contains exceptions that allow web browsers to diverge from the normal RegExp grammar. These exceptions aren't allowed when the "unicode" flag is present, but our current implementation doesn't take this into account.

There are many exceptions, but a few examples would be:

  • PatternCharacter: {
  • CharacterClassEscape: [\d-a]
  • DecimalEscape: \3 when the number of capturing parentheses is less than 3
  • IdentityEscape: \h, \u
  • QuantifiableAssertion Quantifier: (?!.)+

The above should throw a SyntaxError when used as, e.g.,/{/u

Contributor Guide

Research-Richtung
Überprüfen Sie, wie das Unicode Flag im Regex Parsing Code erkannt wird, und stellen Sie sicher, dass die Ausnahmen von Annex B.1.4 übersprungen werden, wenn dieses Flag gesetzt ist. Suchen Sie nach den Bereichen, in denen PatternCharacter, CharacterClassEscape, DecimalEscape, IdentityEscape und QuantifiableAssertion Quantifier geparst werden, und fügen Sie bedingte Prüfungen für das Unicode Flag hinzu.
Tech Stack
javascript
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
JavaScriptRegExp
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30