04-29-2014, 05:02 AM
I strongly believe that all functions except the command and hook handlers, and the local callbacks, should have a comment saying what the function does, what params it takes and what the return values are. And it should verify its params using assert(). It would have to be a very special-case function to make me consider otherwise.