Catch2 icon indicating copy to clipboard operation
Catch2 copied to clipboard

Add -Wsign-conversion to the list of common warning flags

Open Wmbat opened this issue 2 years ago • 2 comments

Description

Fixes all implicit sign conversions and turns on -Wsign-conversion when building catch2. I usually tried to find a good solution to the warning instead of casting.

Ran all the tests on my machine and it seemed good to go

GitHub Issues

Related to #2583

Wmbat avatar Dec 06 '23 22:12 Wmbat

Codecov Report

Merging #2775 (f28ab91) into devel (0520ff4) will decrease coverage by 0.06%. The diff coverage is 75.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2775      +/-   ##
==========================================
- Coverage   91.45%   91.39%   -0.06%     
==========================================
  Files         194      194              
  Lines        8197     8196       -1     
==========================================
- Hits         7496     7490       -6     
- Misses        701      706       +5     

codecov[bot] avatar Dec 07 '23 00:12 codecov[bot]