Uh

It’s funny the combination of paranoias you can find in any one person. The same coder who apparently shunned the concept of checking to see if the arguments being passed him were valid has big blocks of code that do nothing but populate structures and then check to see if they still look valid. Maybe he couldn’t figure out why his code was crashing someplace else, so he decided to make sure his data was valid *before* he called all his code?

This is the kind of code you should comment, people:

extern int wrapperFunc(int somearg, int anotherarg, class& otherarg, class* pointer)
{
  if(pointer == NULL)
    otherFunc(pointer);
}

Now, when I came across this code… It did have a comment…

extern int wrapperFunc(int somearg, int anotherarg, class& otherarg, class* pointer)
{
  if(pointer == NULL)
    // Call otherFunc with value of pointer
    otherFunc(pointer);
}

Dude!

You are so fired.

It’s a good job none of this code is actually in use… :)