tc39/proposal-numeric-separator

Need to update grammar production in Annex B

waldemarhorwat opened this issue · 5 comments

waldemarhorwat commented

Annex B has an alternate definition of DecimalIntegerLiteral. The proposal should propagate the change to the DecimalIntegerLiteral production (and only that one production) to the Annex B version.

The intent is that octal and other weird legacy literals won't be allowed to contain underscores, but this update is necessary to allow underscores after the first digit in plain decimal literals.

rwaldron commented

Will do, thanks for spotting and reporting

rwaldron commented

@waldemarhorwat here's a screen shot of the specific addition:

  • Ensures that NumericLiteralSeparator is allowed after first digit in Decimal Literals
  • Does not add grammar for any legacy literals
anba commented

Just to clarify: 08_1 should be disallowed, but 08.1_2 and 08e1_2 will be allowed?

rwaldron commented

08_1 should be disallowed

Yes

08.1_2 and 08e1_2 will be allowed?

Yes, because:

. DecimalDigits ExponentPart

anba commented

Okay, thanks for the clarification!