Skip to content
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

Wextra #125

Merged
merged 2 commits into from
Aug 19, 2024
Merged

Wextra #125

merged 2 commits into from
Aug 19, 2024

Commits on Aug 19, 2024

  1. Add extra warnings 

    The main things these picked up are fall-throughs on switch
statements, all of which were deliberate.  However they're now
annotated as such so no longer complain.  In samtools this did detect
a genuine bug so it's worth adding (and annotating to silence FPs)
    @jkbonfield
    jkbonfield committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    bbbe4ee View commit details
    Browse the repository at this point in the history

  2. Updated NEWS with changes so far (including this PR). 

    Not strictly part of the same change, but it's all necessary prior to
the next release and neither commit is a change of code.
    @jkbonfield
    jkbonfield committed Aug 19, 2024
    Configuration menu
    Copy the full SHA
    04979a7 View commit details
    Browse the repository at this point in the history