-
Notifications
You must be signed in to change notification settings - Fork 153
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Tweak discussion of unicode; link sections together.
- Loading branch information
Showing
1 changed file
with
6 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 comments
on commit 902ff45
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mnot Looks good, except that I feel the need for an article in "if they occur in input_sequence, serialization will fail."
I think both "if they occur in the input_sequence, serialization will fail." and "if they occur in an input_sequence, serialization will fail." should be fine; what's best may depend on context.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's the style used throughout the rest of the spec, for brevity.
Nit: we probably should explain the "U+" notation. I did that in an earlier commit that never made it into a PR (see cbfba24). That change also had examples for non-serializible strings.