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

Jump to bottom

clean up [algorithms] source code #2509

Closed
zygoloid opened this issue Nov 26, 2018 · 2 comments · Fixed by #2547
Closed

clean up [algorithms] source code #2509

zygoloid opened this issue Nov 26, 2018 · 2 comments · Fixed by #2547
Assignees
@jensmaurer
Labels
big An issue causing a large set of changes, scattered across most of the text.

Comments

@zygoloid
Copy link
Member

From @jensmaurer in #2501:

The "algorithms" clause still has a lot of troff-era formatting where we start a new line for each non-prose item, such as \tcode{first}. And we have a lot of these. This makes for very long source code. Suggestion: Between the mailings, have me reformat the source code in a no-visual-changes diff.
@zygoloid zygoloid mentioned this issue Nov 26, 2018
Closed
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 26, 2018

Yes please, this sounds like a big improvement of the quality of life.

@CaseyCarter
Copy link
Contributor

Having recently spent a lot of time in [algorithms], +1. The "vertical" layout makes the source less readable, and makes it painful to correlate the source with the rendered text.

@jensmaurer jensmaurer added the big An issue causing a large set of changes, scattered across most of the text. label Nov 27, 2018
@jensmaurer jensmaurer self-assigned this Dec 1, 2018
@jensmaurer jensmaurer mentioned this issue Dec 2, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
big An issue causing a large set of changes, scattered across most of the text.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[algorithms] Compactify and reformat source with no visual change jensmaurer/draft
4 participants
@zygoloid @CaseyCarter @tkoeppe @jensmaurer