When contracting, I get to see lots of code. Some of it is good, some of it bad, and some just leaves me scratching my head.

For example, I came across this today:

//ASSERT(1);

Apparently things had gotten so bad for this programmer that he needed to make sure 1 still evaluated to true. I’ve had days like that. Better yet, when done, he just commented the line out, helpfully allowing future developers to quickly make use of his wisdom.

Such a philanthropist.