Skip to content

Editorial: fix some minor editorial convention violations#3805

Merged
ljharb merged 1 commit intomainfrom
editorial-convention-violations
Apr 24, 2026
Merged

Editorial: fix some minor editorial convention violations#3805
ljharb merged 1 commit intomainfrom
editorial-convention-violations

Conversation

@michaelficarra
Copy link
Copy Markdown
Member

@michaelficarra michaelficarra commented Apr 8, 2026

Found these using some in-development ecmarkup lint rules.

edit: tc39/ecmarkup#693

Comment thread spec.html
1. If _E_ and _D_ are not the same Shared Data Block event, then
1. If it is not the case that both _E_ happens-before _D_ in _execution_ and _D_ happens-before _E_ in _execution_, then
1. If _E_ and _D_ are both WriteSharedMemory or ReadModifyWriteSharedMemory events and _E_ and _D_ do not have disjoint memory ranges, then
1. If _E_ is either a WriteSharedMemory or ReadModifyWriteSharedMemory event, _D_ is either a WriteSharedMemory or ReadModifyWriteSharedMemory event, and _E_ and _D_ do not have disjoint memory ranges, then
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think this one actually read ambiguously and I'm really happy to be getting rid of it!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

The rendered spec preview for this PR is available as a single page at https://tc39.es/ecma262/pr/3805 and as multiple pages at https://tc39.es/ecma262/pr/3805/multipage .

Comment thread spec.html Outdated
@nicolo-ribaudo nicolo-ribaudo added the ready to merge Editors believe this PR needs no further reviews, and is ready to land. label Apr 17, 2026
@ljharb ljharb force-pushed the editorial-convention-violations branch from 58c55e5 to 6d0cff6 Compare April 24, 2026 01:45
@ljharb ljharb merged commit 6d0cff6 into main Apr 24, 2026
10 checks passed
@ljharb ljharb deleted the editorial-convention-violations branch April 24, 2026 01:54
jmdyck added a commit to jmdyck/ecma262 that referenced this pull request Apr 28, 2026
PR tc39#3789 overwrote a wording change from PR tc39#3805.
ljharb pushed a commit to jmdyck/ecma262 that referenced this pull request Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial change ready to merge Editors believe this PR needs no further reviews, and is ready to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants