Skip to content

Commit 6e3f2d3

Browse files
authored
Add note on containing warning disabling code.
1 parent 6445e67 commit 6e3f2d3

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

02-Use_the_Tools_Available.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,8 @@ Both of these tools use coverage reporting to find new code execution paths and
244244

245245
If it is determined by team consensus that the compiler or analyzer is warning on something that is either incorrect or unavoidable, the team will disable the specific error to as localized part of the code as possible.
246246

247+
Be sure to reenable the warning after disabling it for a section of code. You do not want your disabled warnings to [leak into other code](http://www.forwardscattering.org/post/48).
248+
247249
## Testing
248250

249251
CMake, mentioned above, has a built in framework for executing tests. Make sure whatever build system you use has a way to execute tests built in.

0 commit comments

Comments
 (0)