G
Gerry Quinn
As a code reviewer, I'd accuse you of inconsistent style. Why do you
explicitly check buf against 0, but implicitly check fp? (Personally I
prefer to _always_ explicitly check, and I prefer using the NULL macro to
help reinforce that I'm dealing with a pointer and not an int).
I'd accept the accusation! I was trying to keep it similar to the
original, but I did it both ways for some reason - not sure why. I also
use NULL, but I thought the implicit zero might be a C thing. As I
said, I don't use that language.
- Gerry Quinn