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 opened on 2016年3月10日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)JavaScript (1,374 forks)batch import
BugSeverity: 2help wanted

Repository metrics

Stars
 (9,000 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

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

コントリビューターガイド

調査方針
ユニコードフラグが正規表現解析コードでどのように検出されるかを確認し、そのフラグが設定されている場合にAnnex B.1.4の例外をスキップするようにします。PatternCharacter、CharacterClassEscape、DecimalEscape、IdentityEscape、QuantifiableAssertion Quantifierが解析される領域を見つけ、ユニコードフラグの条件チェックを追加します。
技術スタック
javascript
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
JavaScriptRegExp
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
30