Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc editorial #1053

Merged
merged 48 commits into from
Feb 13, 2018
Merged

Misc editorial #1053

Changes from 1 commit
Commits
Show all changes
48 commits
Select commit Hold shift + click to select a range
8df2809
Editorial: add more lookahead-restrictions to NotEscapeSequence
jmdyck Dec 21, 2017
c564154
Editorial: insert missing </emu-eqn>
jmdyck Jan 4, 2018
dbf7751
Editorial: delete extraneous dot
jmdyck Jan 4, 2018
cf8c857
Editorial: delete extra spaces and add a dot
jmdyck Jan 4, 2018
d7f1c6c
Editorial: add '*' around 'true' and 'false'
jmdyck Jan 4, 2018
12dc570
Editorial: fix typo "algorthm"
jmdyck Jan 4, 2018
0f1d570
Editorial: resolve "LocalTimeZoneAdjustment" vs "LocalTZA" inconsistency
jmdyck Jan 4, 2018
ec1cd8c
Editorial: Remove '_' RHS for IdentifierPart
jmdyck Jan 5, 2018
b4081da
Editorial: entity-encode < and >
jmdyck Jan 24, 2018
0e04451
Editorial: tweak some indentation
jmdyck Jan 24, 2018
342fdb0
Editorial: consistify grammar params in defining prodns
jmdyck Jan 24, 2018
3c8d7fc
Editorial: elide unnecessary grammar params in non-defining prodns
jmdyck Jan 24, 2018
7c9b35e
Editorial: fix mentions of Atom :: `(` Disjunction `)`
jmdyck Jan 24, 2018
0a3b5a8
Editorial: "enclosing RegExp" -> "enclosing |Pattern|"
jmdyck Jan 24, 2018
2ac876f
Editorial: insert hyphen in "left-capturing"
jmdyck Jan 24, 2018
076284d
Editorial: "Otherwise" -> "Else"
jmdyck Jan 24, 2018
22e7117
Editorial: move the paren before "U+003E" to after
jmdyck Jan 24, 2018
1f2d1cb
Editorial: invert "Let the enclosed substring be _groupName_"
jmdyck Jan 24, 2018
c478751
Editorial: recap "left-capturing parentheses" more concisely.
jmdyck Jan 24, 2018
9a455bc
Editorial: delete "this following"
jmdyck Jan 24, 2018
63b6d27
Editorial: move StringValue-for-RegExpIdentifierName clause
jmdyck Jan 24, 2018
956e094
Editorial: move BackreferenceMatcher to a more appropriate location
jmdyck Jan 24, 2018
577f76f
Editorial: "Abstract Operation" -> "( _n_ )" in header
jmdyck Jan 24, 2018
f5cddb2
Editorial: add _namedCaptures_ parameter to GetSubstitution()
jmdyck Jan 24, 2018
7b2fe11
Editorial: "|Term|" -> "the located |GroupSpecifier|"
jmdyck Jan 24, 2018
261bc18
Editorial: "Let" -> "Set" for already-bound metavariable
jmdyck Jan 24, 2018
a1f915b
Editorial: remove `_` RHS from RegExpIdentifierPart
jmdyck Jan 24, 2018
f09e801
Editorial: tweak indentation
jmdyck Jan 24, 2018
35ab8d5
Editorial: move PromiseResolve clause into Promise.resolve clause
jmdyck Jan 24, 2018
feaa751
Editorial: add "then" to "If" step
jmdyck Jan 24, 2018
cd12b08
Markup: entity-encode &ldquo; and &rdquo;
jmdyck Jan 25, 2018
57b25af
Editorial: right-single-quotation-mark -> apostrophe twice
jmdyck Jan 25, 2018
f2ed860
Markup: <emu-nt>Foo</emu-nt> -> |Foo|
jmdyck Jan 25, 2018
0fc5f71
Editorial: Add [?U] to RHS occurrences of CharacterClassEscape
jmdyck Jan 25, 2018
2ad9c5f
Editorial: delete extraneous backslashes
jmdyck Jan 25, 2018
2a7bb57
Editorial: `General_Category` -> &ldquo;General_Category&rdquo;
jmdyck Jan 25, 2018
6a5870c
Editorial: |True| -> &ldquo;True&rdquo;
jmdyck Jan 25, 2018
392b8d9
Editorial: move a sentence about property values
jmdyck Jan 25, 2018
607e7d1
Editorial: tighten up the "must support/must not support" wording
jmdyck Jan 25, 2018
9a4fa7a
Editorial: Put algorithm before supporting tables
jmdyck Jan 25, 2018
e3d9eff
Editorial: reorder UnicodePropertyValueExpression-related prodns
jmdyck Jan 25, 2018
0d79be5
Editorial: tweak UnicodePropertyValueExpression eval algs
jmdyck Jan 25, 2018
b756cef
Editorial: avoid reusing/overwriting metavariable _p_
jmdyck Jan 25, 2018
d26d24a
Editorial: misc tweaks re lookbehind assertions
jmdyck Jan 27, 2018
6e04f06
Editorial: add _direction_ to BackreferenceMatcher's param list
jmdyck Jan 27, 2018
03bd376
Markup: fix well-formedness error
jmdyck Feb 8, 2018
17da467
Markup: tweak note re parsing the same String multiple times.
jmdyck Feb 8, 2018
528eb61
Markup: insert <p>...</p> tags
jmdyck Feb 8, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 11 additions & 11 deletions spec.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29721,7 +29721,7 @@ <h1>Pattern</h1>
<!-- es6num="21.2.2.3" -->
<emu-clause id="sec-disjunction">
<h1>Disjunction</h1>
<p>With argument _direction_.</p>
<p>With parameter _direction_.</p>
<p>The production <emu-grammar>Disjunction :: Alternative</emu-grammar> evaluates as follows:</p>
<emu-alg>
1. Evaluate |Alternative| to obtain a Matcher _m_.
Expand Down Expand Up @@ -29752,7 +29752,7 @@ <h1>Disjunction</h1>
<!-- es6num="21.2.2.4" -->
<emu-clause id="sec-alternative">
<h1>Alternative</h1>
<p>With argument _direction_.</p>
<p>With parameter _direction_.</p>
<p>The production <emu-grammar>Alternative :: [empty]</emu-grammar> evaluates as follows:</p>
<emu-alg>
1. Return a Matcher that takes two arguments, a State _x_ and a Continuation _c_, and returns the result of calling _c_(_x_).
Expand All @@ -29768,7 +29768,7 @@ <h1>Alternative</h1>
1. Else,
1. Assert: _direction_ is equal to -1.
1. Return an internal Matcher closure that takes two arguments, a State _x_ and a Continuation _c_, and performs the following steps when evaluated:
1. Let _d_ be a Continuation that takes a State argument _y_ and returns the result of calling _m1_(_y_, _c_)
1. Let _d_ be a Continuation that takes a State argument _y_ and returns the result of calling _m1_(_y_, _c_).
1. Call _m2_(_x_, _d_) and return its result.
</emu-alg>
<emu-note>
Expand All @@ -29779,7 +29779,7 @@ <h1>Alternative</h1>
<!-- es6num="21.2.2.5" -->
<emu-clause id="sec-term">
<h1>Term</h1>
<p>With argument _direction_.</p>
<p>With parameter _direction_.</p>
<p>The production <emu-grammar>Term :: Assertion</emu-grammar> evaluates as follows:</p>
<emu-alg>
1. Return an internal Matcher closure that takes two arguments, a State _x_ and a Continuation _c_, and performs the following steps when evaluated:
Expand Down Expand Up @@ -30265,7 +30265,7 @@ <h1>Quantifier</h1>
<!-- es6num="21.2.2.8" -->
<emu-clause id="sec-atom">
<h1>Atom</h1>
<p>With argument _direction_.</p>
<p>With parameter _direction_.</p>
<p>The production <emu-grammar>Atom :: PatternCharacter</emu-grammar> evaluates as follows:</p>
<emu-alg>
1. Let _ch_ be the character matched by |PatternCharacter|.
Expand Down Expand Up @@ -30298,12 +30298,12 @@ <h1>Atom</h1>
1. Let _xe_ be _x_'s _endIndex_.
1. Let _ye_ be _y_'s _endIndex_.
1. If _direction_ is equal to +1, then
1. Assert: _xe_ &lte; _ye_.
1. Let _s_ be a fresh List whose characters are the characters of _Input_ at indices _xe_ (inclusive) through _ye_ (exclusive).
1. Assert: _xe_ &le; _ye_.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the intention here was less than or equal to. While gte/lte are not entities, &le;= would preserve the intent. Is < a better assert? /cc @littledan

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

never mind I don't know HTML entities.

1. Let _s_ be a new List whose elements are the characters of _Input_ at indices _xe_ (inclusive) through _ye_ (exclusive).
1. Else,
1. Assert: _direction_ is equal to -1.
1. Assert: _ye_ &lte; _xe_.
1. Let _s_ be a fresh List whose characters are the characters of _Input_ at indices _ye_ (inclusive) through _xe_ (exclusive).
1. Assert: _ye_ &le; _xe_.
1. Let _s_ be a new List whose elements are the characters of _Input_ at indices _ye_ (inclusive) through _xe_ (exclusive).
1. Set _cap_[_parenIndex_+1] to _s_.
1. Let _z_ be the State (_ye_, _cap_).
1. Call _c_(_z_) and return its result.
Expand All @@ -30322,7 +30322,7 @@ <h1>Runtime Semantics: CharacterSetMatcher ( _A_, _invert_, _direction_ )</h1>
1. Return an internal Matcher closure that takes two arguments, a State _x_ and a Continuation _c_, and performs the following steps when evaluated:
1. Let _e_ be _x_'s _endIndex_.
1. Let _f_ be _e_ + _direction_.
1. If _f_ &lt; 0 or _f_ > _InputLength_, return ~failure~.
1. If _f_ &lt; 0 or _f_ &gt; _InputLength_, return ~failure~.
1. Let _index_ be min(_e_, _f_).
1. Let _ch_ be the character _Input_[_index_].
1. Let _cc_ be Canonicalize(_ch_).
Expand Down Expand Up @@ -30424,7 +30424,7 @@ <h1>Runtime Semantics: UnicodeMatchPropertyValue ( _p_, _v_ )</h1>
<!-- es6num="21.2.2.9" -->
<emu-clause id="sec-atomescape">
<h1>AtomEscape</h1>
<p>With argument _direction_.</p>
<p>With parameter _direction_.</p>
<p>The production <emu-grammar>AtomEscape :: DecimalEscape</emu-grammar> evaluates as follows:</p>
<emu-alg>
1. Evaluate |DecimalEscape| to obtain an integer _n_.
Expand Down