There is a colleague of mine who constantly writes:

if (someBool == true)


It drives me up the wall!  Should I make a big deal of it or just drop it?

It's only redundant code, not life or death.  However....

If it's happening a lot, it could be a problem with how someBool is being named. A good name can go a long way towards eliminating the need for the ==true

if(IsSomeCondition)


or

if(hasCondition)


or

if(somethingExists)


for example.
