Add GCC 15 support, excluding CI

[laurynas-biveinis]

• Fix a build warning
• Bump GCC version in CMakePresets.json

CI is excluded because of no simple way to get GCC 15 on Ubuntu 24.04 (or 22.04)

Summary by CodeRabbit

• Chores

  • Updated build configuration to target newer compiler versions.

• Tests

  • Refined test code for encoding and decoding verification.

[coderabbitai]

Walkthrough

Version bump for GCC compiler targets from 14 to 15 in build configuration, and simplification of a test parameter by removing an unnecessary explicit cast from the bitwise complement operation.

Changes

Cohort / File(s)	Summary
Compiler Version Update CMakePresets.json	Updated gcc preset's cache variables: CMAKE_C_COMPILER and CMAKE_CXX_COMPILER bumped from versions 14 to 15
Test Parameter Simplification test/test_key_encode_decode.cpp	Removed explicit static_cast<T> from test argument; changed static_cast<T>(~0ULL) to ~0ULL in UInt64C00010 test

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Both changes are straightforward—one is a direct version increment in configuration, the other is a redundant cast removal that doesn't alter test semantics.

Poem

🔨 From fourteen to fifteen, the compilers rise,
A cast removed—simpler code, no disguise,
Tools refreshed, tests cleaned—tiny sparks of care,
Building stronger, lighter, with time to spare. ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: updating GCC compiler versions from 14 to 15 in CMakePresets.json, with a note about excluding CI changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch gcc-15-minimal

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 351bbbc and 5e25981.

📒 Files selected for processing (2)

• CMakePresets.json (1 hunks)
• test/test_key_encode_decode.cpp (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)

• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Codacy Static Code Analysis

🔇 Additional comments (1)

test/test_key_encode_decode.cpp (1)

351-351: LGTM! Removes redundant cast for uint64_t.

The removal of static_cast<T> where T is std::uint64_t is correct. Since ~0ULL already has type unsigned long long (typically 64 bits), casting it to uint64_t is redundant when they're the same size. This properly addresses the GCC 15 warning mentioned in the PR description.

Note that similar casts at lines 199, 247, and 297 are retained correctly because they perform actual narrowing conversions for smaller types (uint8_t, uint16_t, uint32_t).

[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