Skip to content

Conversation

@bobtista
Copy link

@bobtista bobtista commented Dec 5, 2025

Container Empty Checks (readability-container-size-empty)

Before: if (container.size() == 0)
After: if (container.empty())

Explicit String Comparisons (bugprone-suspicious-string-compare)

Before: if (strcmp(a, b))
After: if (strcmp(a, b) == 0)

Applied automatically using clang-tidy checks:

  • readability-container-size-empty
  • bugprone-suspicious-string-compare

@bobtista bobtista self-assigned this Dec 5, 2025
@bobtista bobtista force-pushed the bobtista/refactor/clang-tidy-readability branch 2 times, most recently from b581c27 to 7e8e0d7 Compare December 6, 2025 01:55
@bobtista bobtista force-pushed the bobtista/refactor/clang-tidy-readability branch from 7e8e0d7 to 9c3661b Compare December 6, 2025 02:00
@xezon
Copy link

xezon commented Dec 6, 2025

Can you please split this into 2 pulls? One for each tidy action.

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.

2 participants