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

[lex] Reorder subclauses to better follow phases of translation #7316

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

AlisdairM
Copy link
Contributor

This PR purely moves existing words around, and does not create any new content.

The proposed subclause ordering is now:

  • 5 Lexical convensions
    • 5.1 Separate translation
    • 5.2 Phases of translation
    • 5.3 Characters
      • 5.3.1 Character sets
      • 5.3.2 Universal character names
    • 5.4 Comments
    • 5.5 Preprocessing tokens
    • 5.6 Header names
    • 5.7 Preprocessing numbers
    • 5.8 Operators and punctuators
    • 5.9 Alternative tokens
    • 5.10 Tokens
    • 5.11 Identifiers
    • 5.12 Keywords
    • 5.13 Literals
      • 5.13.1 Kinds of literals
      • 5.13.2 ...

This PR purely moves existing words around, and does not create any new content.

The proposed subclause ordering is now:

* 5 Lexical convensions
  - 5.1 Separate translation
  - 5.2 Phases of translation
  - 5.3 Characters
    - 5.3.1 Character sets
    - 5.3.2 Universal character names
  - 5.4 Comments
  - 5.5 Preprocessing tokens
  - 5.6 Header names
  - 5.7 Preprocessing numbers
  - 5.8 Operators and punctuators
  - 5.9 Alternative tokens
  - 5.10 Tokens
  - 5.11 Identifiers
  - 5.12 Keywords
  - 5.13 Literals
    - 5.13.1 Kinds of literals
    - 5.13.2 ...
@AlisdairM
Copy link
Contributor Author

This is the clean version of the original PR #7193, with no changes other than strictly reordering the subclauses.

@tkoeppe tkoeppe merged commit 801fb2c into cplusplus:main Oct 17, 2024
2 checks passed
@AlisdairM AlisdairM deleted the reorder_lex branch October 18, 2024 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants