> No, I don't think this is obvious. What does it mean to have a null > string here and how can it happen? I'm pretty sure it can't, and the > if check is just clutter. The reasoning is that, since we checked it for NULL in the first statement of the function, we must believe that the possibility exists for it to be NULL.