Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Make catch.hpp a system header to avoid compilation warnings #400

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dawagner
Copy link
Contributor

@dawagner dawagner commented Jan 24, 2017

catch.hpp may contain warnings that are turned to error by -Werror; this
will cause the compilation to fail unless catch.hpp is a "system"
header. Tweak the CMakefile to make it so.

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

Merging #400 into master will not change coverage

@@             master       #400   diff @@
==========================================
Files           214        214
Lines          6674       6674
Methods        1447       1447
Messages          0          0
Branches        845        845
==========================================
Hits           4825       4825
Misses         1382       1382
Partials        467        467

Powered by Codecov. Last update d55a88f...14c9b60

  

catch.hpp may contain warnings that are turned to error by -Werror; this
will cause the compilation to fail unless catch.hpp is a "system"
header. Tweak the CMakefile to make it so.

Signed-off-by: David Wagner <david.wagner@intel.com>
@codecov-io
Copy link

Current coverage is 72.29% (diff: 100%)

Merging #400 into master will not change coverage

@@             master       #400   diff @@
==========================================
  Files           214        214          
  Lines          6674       6674          
  Methods        1447       1447          
  Messages          0          0          
  Branches        845        845          
==========================================
  Hits           4825       4825          
  Misses         1382       1382          
  Partials        467        467          

Powered by Codecov. Last update d55a88f...14c9b60

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants