Skip to content

Normalize the language specification whitespace #4335

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

eernstg
Copy link
Member

@eernstg eernstg commented Apr 24, 2025

This PR makes no semantic changes to the language specification, but it changes many details about the whitespace and a couple of other formatting properties.

\commentary{} and \rationale{} blocks are indented consistently (that is, they indent the body by 2 spaces, which was previously only the case in some locations).

An empty line occurs before \begin{itemize} and before \begin{enumerate}, consistently.

In parenthesized commentary, the parentheses are consistently shown in commentary font (dimmed). This means that (\commentary{...}) was changed to \commentary{(...)} in a number of locations.

Added/removed a missing/extra space (...}or ... --> ...} or ... and which is --> which is, some double spaces after a period or colon).

A non-trivial number of locations may seem to have been modified because a line break was moved (typically because some lines were too long after indentation).

@eernstg eernstg requested a review from lrhn April 24, 2025 13:30
@eernstg eernstg force-pushed the normalize_spec_whitespace_apr25 branch from d0b9fb2 to 183ca95 Compare April 24, 2025 13:33
@eernstg
Copy link
Member Author

eernstg commented Apr 29, 2025

Note that reviewing can be done with a "Hide whitespace" setting which will reduce the number of lines in the diff very substantially.

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.

1 participant