Hacker News new | ask | show | jobs
by dataflow 480 days 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.)

1 comments

Yep, plus static analysis in CI. We also run ubasan, tsan, and valgrind on our unit tests.