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

Examples cleanup #1449

Merged
merged 26 commits into from
Jul 30, 2021
Merged

Examples cleanup #1449

merged 26 commits into from
Jul 30, 2021

Conversation

julianoes
Copy link
Collaborator

@julianoes julianoes commented May 29, 2021

Taking a pass through all examples.

This includes:

  • Naming improvements of the examples
  • Replacing std::endl with \n (cause flushing is not always required, in general)
  • Removing log output colors (I want to focus on simple code that's easy to read, rather than pretty output, and in the future we might make use of std::fmt anyway for that.)
  • Replace most custom helpers with more boilerplate but easier to grasp repetitive code.
  • Clean up file comments (remove filename in comment and remove authors, as this tends to be always out of date and wrong)

@julianoes julianoes marked this pull request as ready for review July 29, 2021 11:49
@julianoes julianoes merged commit 5c03390 into main Jul 30, 2021
@julianoes julianoes deleted the pr-example-cleanup branch July 30, 2021 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant