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 re recent commits #1023

Merged
merged 6 commits into from
Oct 25, 2017
Merged

Misc Editorial re recent commits #1023

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b5794dd
Editorial: fix 2 typos from recent commit
jmdyck Oct 16, 2017
296f6e0
Editorial: Add blank lines in multi-prodn <emu-grammar>s
jmdyck Oct 22, 2017
8de9d52
Editorial: Un-expand a needlessly expanded prodn
jmdyck Oct 22, 2017
0933cf9
Editorial: Change [U] to <sub>[U]</sub>
jmdyck Oct 22, 2017
30a37ea
Editorial: "code unit value" -> "numeric value"
jmdyck Oct 22, 2017
81c4c42
Editorial: Change "the code point value of the SV of |HexEscapeSequen…
jmdyck Oct 22, 2017
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
2 changes: 1 addition & 1 deletion spec.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29418,7 +29418,7 @@ <h1>Static Semantics: CharacterValue</h1>
</emu-note>
<emu-grammar>CharacterEscape :: HexEscapeSequence</emu-grammar>
<emu-alg>
1. Return the code point value of the SV of |HexEscapeSequence|.
1. Return the numeric value of the code unit that is the SV of |HexEscapeSequence|.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think I picked code point value, so I could use the same phrase within the complete section (21.2.1.4 Static Semantics: CharacterValue). Only changing it to numeric value of the code unit for CharacterEscape :: HexEscapeSequence looks a bit odd, because also other productions in 21.2.1.4 evaluate to a single code unit.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

other productions in 21.2.1.4 evaluate to a single code unit.

No, CharacterValue returns an integer in the Unicode codespace, which is too big for a single (16-bit) code unit.

Elsewhere in 21.2.1.4, in occurrences of "the code point value of X" (or "X's code point value"), X either literally denotes a code point (e.g. "U+002D (HYPHEN-MINUS)") or is a metavariable whose value is a code point. But the SV of |HexEscapeSequence| is not a code point, so I think it would be misleading to use the same formulation.

</emu-alg>
<emu-grammar>RegExpUnicodeEscapeSequence :: `u` LeadSurrogate `\u` TrailSurrogate</emu-grammar>
<emu-alg>
Expand Down