01org / parameter-framework / 1310

32%

Build Type

Pull #310

travis-ci

Committed by David Wagner

Commit Message

Add a CMake option to not turn warnings into errors

We have some warnings in Release configuratio because of:
- unused arguments (some calls to `assert`, that are completly removed in
  release mode, cause some arguments to be unused) in such a pattern:

    void f(bool b)
    {
        assert(b);
    }
- strict aliasing rules violation in floating point tests. Since this is only
  test code, it is not considered critical.

Add an option (on by default) to add the -Werror flag. Have travis travis build
the Parameter Framework in "Release" configuration (it previously built in
"Debug" and "None" configurations) without the -Werror flag. Also, forcefully
remove the "/WX" flag on windows (equivalent to -Werror) because we are not
warning-free yet, even in debug mode.

Signed-off-by: David Wagner <david.wagner@intel.com>

Pull Request Pull Request #310:

Run Details

10 of 10 new or added lines in 2 files covered. (100.0%)

4995 of 6670 relevant lines covered (74.89%)

3712.46 hits per line