Fix CI build warnings #322

Activity

1. wgtmac

   added

   good first issueGood for newcomersGood for newcomers

   on Nov 15, 2025
2. wgtmac

   changed the title [-]Fix build warnings[/-] [+]Fix CI build warnings[/+] on Nov 15, 2025
3. Jinchul81 commented on Nov 19, 2025

   

   Jinchul81

   on Nov 19, 2025

   More actions

   I am interested in this work as a newcomer. May I provide a PR?
4. Fokko commented on Nov 19, 2025

   

   Fokko

   on Nov 19, 2025

   Contributor

   More actions

   @Jinchul81 Go ahead, first come, first serve :)
5. Jinchul81 commented on Nov 19, 2025

   

   Jinchul81

   on Nov 19, 2025

   More actions

   Please review the PR #328
6. Joe-Abraham commented on Mar 8, 2026

   

   Joe-Abraham

   on Mar 8, 2026

   Last edited by Joe-Abraham

   More actions

   I've created a PR to resolve the build warnings reported here. The fixes have been split into logical commits to make the review process easier:

   1. Fixed unused parameter warnings: Added [[maybe_unused]] where appropriate.
   2. Fixed uninitialized member warnings: Added default initializers to struct and class members.
   3. Fixed integer sign comparison warnings: Used <utility> functions like std::cmp_equal, std::cmp_less.
   4. Fixed various other build warnings: Addressed specific operator precedence warnings and redundant std::move calls.
   5. Cleaned up shadowing warnings/unused functions: Renamed shadowed arguments and removed unused local test helpers.

   PR Link - #587

   @Fokko @wgtmac could some one help review and merge
7. ankitflow001

   added 2 commits that reference this issue on Nov 1, 2026

   fix: suppress -Wunused-parameter and -Wshadow warnings under -Wextra

   ...

   This issue will close once commit 180f0bf is merged into the 'main' branch.

   180f0bf

   fix: suppress -Wunused-parameter and -Wshadow warnings under -Wextra

   ...

   This issue will close once commit 3ba2624 is merged into the 'main' branch.

   3ba2624
8. wgtmac

   closed this as completedin #1081on Nov 3, 2026