>I don't see how that applies. I advised him not to use empty at all. If there is a tab character in a field, I would not consider it empty.
>
Existing code is always correct at first approximation - the code you advised would change the biz logic of the existing program.
Such changes should be mentioned when advising alternate coding.