Skip to content

[mini] fix minor warnings found via clang-tidy #466

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
Merged
SeverinDiederichs merged 1 commit into from
Jan 9, 2026
Merged

[mini] fix minor warnings found via clang-tidy #466

SeverinDiederichs merged 1 commit into from
Jan 9, 2026

Conversation

@SeverinDiederichs
Copy link
Collaborator

@SeverinDiederichs SeverinDiederichs commented Jan 7, 2026

This mini PR fixes a few warnings reported by clang tidy for

bugprone-signed-char-misuse
bugprone-unhandled-self-assignment
bugprone-switch-missing-default-case

@SeverinDiederichs SeverinDiederichs added the cleaning improves code clarity, readability or other label Jan 7, 2026
@phsft-bot
Copy link

phsft-bot commented Jan 7, 2026

Can one of the admins verify this patch?

@SeverinDiederichs SeverinDiederichs merged commit a7f0ca7 into apt-sim:master Jan 9, 2026
3 checks passed
@SeverinDiederichs SeverinDiederichs deleted the fix_minor_clangtidy_warnings branch January 9, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JuanGonzalezCaminero JuanGonzalezCaminero JuanGonzalezCaminero approved these changes

Assignees

No one assigned

Labels

cleaning improves code clarity, readability or other

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SeverinDiederichs @phsft-bot @JuanGonzalezCaminero