Menu

#5608 Avoid easy warnings with Clang

Fixed
Maintainability
2019-11-25
2019-11-19
No

Avoid easy warnings with Clang

Building with Clang is currently a lot nosier than with GCC. This
represents a first step into the right direction by picking the
low-hanging fruits.

  1. Fix warning flag detection for Clang

Clang requires -Werror=unknown-warning-option to actually throw an
error if it doesn't know about a warning flag.

  1. Fix struct vs. class template for PQueue

Clang issues a warning that this could be an issue for the
"Windows C++ ABI". Apparently, that's not really causing problems
as the mismatch was introduced by commit c8989177a in 1997...

http://codereview.appspot.com/573270043

Discussion

  • Anonymous

    Anonymous - 2019-11-20
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2019-11-20

    Passes make, make test-baseline and a full make doc

     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2019-11-20
     
  • Anonymous

    Anonymous - 2019-11-21
    • Needs: -->
    • Patch: new --> review
    • Type: --> Enhancement
     
  • Anonymous

    Anonymous - 2019-11-21

    Passes make, make test-baseline and a full make doc

     
  • Anonymous

    Anonymous - 2019-11-23
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-11-23

    Patch on countdown for Nov 25th

     
  • Anonymous

    Anonymous - 2019-11-25
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2019-11-25

    Patch counted down - please push.

     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2019-11-25
     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2019-11-25
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Needs: -->
    • Patch: new -->
    • Type: --> Maintainability
     
  • Jonas Hahnfeld

    Jonas Hahnfeld - 2019-11-25

    Pushed to staging as:

    commit 9926ad3e43ab99c7463ca646fd4321647a9d72da
    Author: Jonas Hahnfeld <hahnjo@hahnjo.de>
    Date:   Tue Nov 19 18:56:26 2019 +0100
    
        Issue 5608/2: Fix struct vs. class template for PQueue
    
        Clang issues a warning that this could be an issue for the
        "Windows C++ ABI". Apparently, that's not really causing problems
        as the mismatch was introduced by commit c8989177a in 1997...
    
    commit 0b62028523d8d81cbc81b48ce10d14d9f680510c
    Author: Jonas Hahnfeld <hahnjo@hahnjo.de>
    Date:   Tue Nov 19 18:55:29 2019 +0100
    
        Issue 5608/1: Fix warning flag detection for Clang
    
        Clang requires -Werror=unknown-warning-option to actually throw an
        error if it doesn't know about a warning flag.