Closed d-b-w closed 4 years ago
Specifically, I ran these checks:
-checks=performance-*,modernize-*,-modernize-use-noexcept,misc-throw-by-value-catch-by-reference,misc-unused-alias-decls,misc-unused-using-decls,google-readability-*,-google-readability-todo,readability-redundant-*,readability-simplify-*
There is also a commit in here that I wrote manually, but which is in the same spirit. The manual commit is swap to use erase/remove instead of a bunch of separate erases. Just something I did while trying to understand code flow.
Specifically, I ran these checks:
-checks=performance-*,modernize-*,-modernize-use-noexcept,misc-throw-by-value-catch-by-reference,misc-unused-alias-decls,misc-unused-using-decls,google-readability-*,-google-readability-todo,readability-redundant-*,readability-simplify-*
There is also a commit in here that I wrote manually, but which is in the same spirit. The manual commit is swap to use erase/remove instead of a bunch of separate erases. Just something I did while trying to understand code flow.