5 ms·

You don't even need static analysis (which they should use too) but just a strict coding style check that enforces curly braces. Should've happened after the fi

• tempodox 9y ago
  I find it jaw-dropping how fashionable cowboy coding still is, especially in C family languages.
  • eru 9y ago
    C-family languages are cowboy languages. The whole point of eg C (and C++)'s undefined behaviour is to allow the compiler to make cowboy assumptions like "this array access will never be out of bounds" or "this signed int will never overflow" without having to prove or even justify them. All in the name of 'efficiency'.
• eru 9y ago
  A coding style check _is_ static analysis. (But that's beating a straw man, of course: with that expanded definition your comment just because "you don't even need anything more than the most primitive forms of static analysis".)
  • emsy 9y ago
    I think the point I was trying to make was: If they don't even check for rudimentary things like coding style, can we expect them to check for more sophisticated sources of issues (e.g. memory corruption).
    • eru 9y ago
      Amen.
• astrange 9y ago
  The Python and Haskell programmers would be upset if you added that pre-commit hook.