Skip to content

hygiene sweep: fabs→std::abs, empty {} ctors/dtors → = default, etc#315

Open
ahcorde wants to merge 1 commit into
ahcorde/rolling/mssied_make_shared_overrridefrom
ahcorde/rolling/hygiene_sweep
Open

hygiene sweep: fabs→std::abs, empty {} ctors/dtors → = default, etc#315
ahcorde wants to merge 1 commit into
ahcorde/rolling/mssied_make_shared_overrridefrom
ahcorde/rolling/hygiene_sweep

Conversation

@ahcorde

@ahcorde ahcorde commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Description

  • fabs→std::abs,
  • verbose typename …::type/::value trait spellings → _t/_v aliases
  • .size() > 0→!empty()
  • push_back(Temp(...))→emplace_back(...)
  • empty {} ctors/dtors → = default
  • [[nodiscard]] on side-effect-free getters

Did you use Generative AI?

Claude Opus 4.7

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested review from asymingt and mjcarroll June 16, 2026 10:23
@ahcorde ahcorde self-assigned this Jun 16, 2026
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