refactor(sonar): grouped fixes

[ReenigneArcher]

Description

Fixes many sonar issues:

cpp:S1238, S3230, S6009, S6188, S7172, S5817, S1185, S1763, S1117, S3608, S936, S5008, S5421, S6225, and S1048

Screenshot

Issues Fixed or Closed

Roadmap Issues

Type of Change

• feat: New feature (non-breaking change which adds functionality)
• fix: Bug fix (non-breaking change which fixes an issue)
• docs: Documentation only changes
• style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
• refactor: Code change that neither fixes a bug nor adds a feature
• perf: Code change that improves performance
• test: Adding missing tests or correcting existing tests
• build: Changes that affect the build system or external dependencies
• ci: Changes to CI configuration files and scripts
• chore: Other changes that don't modify src or test files
• revert: Reverts a previous commit
• BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

• Code follows the style guidelines of this project
• Code has been self-reviewed
• Code has been commented, particularly in hard-to-understand areas
• Code docstring/documentation-blocks for new or existing methods/components have been added or updated
• Unit tests have been added or updated for any new or modified functionality

AI Usage

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[read-the-docs-community]

Documentation build overview

📚 libvirtualhid | 🛠️ Build #33225744 | 📁 Comparing 82f6476 against latest (3b37a9d)

🔍 Preview build

134 files changed · ± 134 modified

±Modified

• annotated.html
• classes.html
• classlvh_1_1ButtonSet-members.html
• classlvh_1_1ButtonSet.html
• classlvh_1_1Gamepad-members.html
• classlvh_1_1Gamepad.html
• classlvh_1_1Keyboard-members.html
• classlvh_1_1Keyboard.html
• classlvh_1_1Mouse-members.html
• classlvh_1_1Mouse.html
• and 124 more...

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.38%. Comparing base (3b37a9d) to head (82f6476).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@ Coverage Diff @@## master #15 +/- ##
==========================================
+ Coverage 87.36% 87.38% +0.01% 
==========================================
Files 10 10 Lines 3317 3322 +5 Branches 1151 1157 +6 ==========================================
+ Hits 2898 2903 +5 - Misses 260 306 +46 + Partials 159 113 -46 

Flag	Coverage Δ
Linux-Clang	69.10% <53.84%> (-0.13%)	⬇️
Linux-GCC	81.92% <96.15%> (+0.21%)	⬆️
Windows-MSVC	84.36% <100.00%> (+0.05%)	⬆️
Windows-MinGW-UCRT64	73.66% <100.00%> (+0.10%)	⬆️
macOS	66.32% <20.00%> (-0.30%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
src/core/runtime.cpp	82.53% <100.00%> (+0.12%)	⬆️
src/core/types.cpp	65.85% <100.00%> (-0.82%)	⬇️
src/include/libvirtualhid/runtime.hpp	100.00% <ø> (ø)
src/platform/linux/uhid_backend.cpp	89.14% <100.00%> (ø)

... and 2 files with indirect coverage changes

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b37a9d...82f6476. Read the comment docs.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud