Suppress more warnings using ToolApplicationSupport::suppressWarnings

bloomberg / clangmetatool

A framework for reusing code in Clang tools

https://bloomberg.github.io/clangmetatool/

Apache License 2.0

119 stars 25 forks source link

Suppress more warnings using ToolApplicationSupport::suppressWarnings #42

Closed dbeer1 closed 5 years ago

dbeer1 commented 5 years ago

Describe your changes The ToolApplicationSupport::suppressWarnings function was not suppressing all warnings. In particular, -Wnarrowing was still turned on.

Testing performed Added unit test, which now passes

dbeer1 commented 5 years ago

CC: @ruoso, @HaltCatchFire