Skip to content

Remove an unreachable return statement - #389

Merged
lemire merged 1 commit into
fastfloat:mainfrom
correctmost:cm/remove-unreachable-return
Jun 9, 2026
Merged

lemire merged 1 commit into
fastfloat:mainfrom
correctmost:cm/remove-unreachable-return

Conversation

@correctmost

Copy link
Copy Markdown
Contributor

The redundant statement was reported by cppcheck.

tests/fixedwidthtest.cpp:69:3: Consecutive return, break, continue, goto or throw statements are unnecessary.

The redundant statement was reported by cppcheck.
@lemire
lemire merged commit 9371986 into fastfloat:main Jun 9, 2026
35 checks passed
@correctmost
correctmost deleted the cm/remove-unreachable-return branch June 9, 2026 15:37
social4hyq pushed a commit to social4hyq/homebrew-core that referenced this pull request Sep 20, 2026
fast_float 8.2.9

Created-by: HarmonybrewBot
Commit-by: HarmonybrewBot
Merged-by: HarmonybrewBot
Description: Created by `brew bump`

---

Created with `brew bump-formula-pr`.<details>
  <summary>release notes</summary>
  <pre>## What's Changed
* Remove an else if statement that is always false by @correctmost in fastfloat/fast_float#390
* Remove an unreachable return statement by @correctmost in fastfloat/fast_float#389
* Fix compile error with gcc 9: use of [[unlikely]] by @biojppm in fastfloat/fast_float#392
* reject non-digit wide code units in uint8/uint16 integer fast path by @sahvx655-wq in fastfloat/fast_float#391

## New Contributors
* @correctmost made their first contribution in fastfloat/fast_float#390
* @sahvx655-wq made their first contribution in fastfloat/fast_float#391

**Full Changelog**: https://github.com/fastfloat/fast_float/compare/v8.2.8...v8.2.9</pre>
  <p>View the full release notes at <a href="https://github.com/fastfloat/fast_float/releases/tag/v8.2.9">https://github.com/fastfloat/fast_float/releases/tag/v8.2.9</a>.</p>
</details>
<hr>

See merge request: Harmonybrew/homebrew-core!11380
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.

3 participants