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.

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

Editorial: remove useless es6num comments #1237

Merged
merged 1 commit into from
Jun 18, 2018
Merged

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Jun 18, 2018

Since these comments don't actually get converted into anchors, they serve no purpose. Let's get rid of them.

@bterlson
Copy link
Member

Alright alright, it's time to dispense with this legacy.

(For those wondering, these IDs were preserved in the source text only, and did not impact the rendering in any way. I used them as bookmarks in the source document since section numbers are computed rather than hard-coded making searching difficult. But they're so badly out of date and half-missing now that it's not super useful).

@bterlson bterlson merged commit 496e172 into tc39:master Jun 18, 2018
@ljharb ljharb deleted the es6num branch June 18, 2018 21:47
ljharb added a commit to ljharb/ecma262 that referenced this pull request Jun 19, 2018
bterlson pushed a commit that referenced this pull request Jun 20, 2018
* Editorial: remove useless es6num comments (#1237)

* Editorial: fix typo in Section 6.4.2 (#1233)

Fix a typo in Section 6.4.2, which concerns the Reference Specification
Type in the definition for a "Super Reference".

represents => represent

Fixes: #1229

* Editorial: remove unused variables in ArraySetLength (#1231)

Remove the unused variable (named `succeeded`) in Steps 16.c.iii-iv. in
Section 9.4.2.4, which describes the abstract operation ArraySetLength.

Fixes: #1230
ljharb pushed a commit to jmdyck/ecma262 that referenced this pull request Mar 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants