Skip to content

Normalize the language specification whitespace#4335

Closed
eernstg wants to merge 2 commits into
mainfrom
normalize_spec_whitespace_apr25
Closed

Normalize the language specification whitespace#4335
eernstg wants to merge 2 commits into
mainfrom
normalize_spec_whitespace_apr25

Conversation

@eernstg
Copy link
Copy Markdown
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
Copy Markdown
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.

@eernstg eernstg force-pushed the normalize_spec_whitespace_apr25 branch from 187d656 to 4d00b52 Compare May 12, 2025 14:20
@eernstg
Copy link
Copy Markdown
Member Author

eernstg commented May 12, 2025

Done in another PR.

@eernstg eernstg closed this May 12, 2025
@eernstg eernstg removed the request for review from lrhn May 12, 2025 14:35
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