4 ms·
The blog post suggests that, probably because it’s never been used, it’s just too noisy to turn on. Arguably every unused variable in the code base is a potent
by pja 1y ago
The blog post suggests that, probably because it’s never been used, it’s just too noisy to turn on.
Arguably every unused variable in the code base is a potential bug like this waiting to chomp on some poor user’s data though, so arguably they should be turning it on & dealing with the consequences?
- 0points 1y ago> so arguably they should be turning it on & dealing with the consequences? Most definitely, yes.
- 0points 1y agoFWIW: I learned in a sibling comment, that this compiler flag doesn't produce a warning in this specific case anyway (but I think it should!). In general (for any programming language, really), I would advocate for enabling warnings and address them as you go along. Because that unlocks the developer feature of being able to react when code changes introduces warnings, as they generally point to unsound code.