3 ms·
> We do have a couple disabled by default in the config, but it's still a lot of warnings A lot, yes, but definitely a lot more than 2 that you still don't hav
by dataflow 2y ago
> We do have a couple disabled by default in the config, but it's still a lot of warnings
A lot, yes, but definitely a lot more than 2 that you still don't have enabled. Might be worth looking into them if you haven't already -- you will definitely disable some of them in the process.
(And I assume you already have clang-tidy, but if not, look into that too.)
- connicpu 2y agoYep, plus static analysis in CI. We also run ubasan, tsan, and valgrind on our unit tests.