I agree with you on the first point, at least for consistency's sake. I just ignored them because I was happy when I finally got my clean compile.

What is wrong with IIf? And UBound? They are really VB6 carry-overs? I thought IIf was just a concise way of writing a simple if-statement. Sometimes I use it, sometimes I don't.