On Sat, Aug 7, 2010 at 17:29, Spencer E. Olson [off-list ref] wrote:
I would have no problem doing that. I was just trying to match the
style already present in http.c. All the other pointer tests in that
function/file are done the same.
Ah, fair enough. I didn't notice that the rest of http.c already used
that style, nevermind then.
It's best to just keep it consistent, and maybe clean it up in some
later patch, but best not to introduce an inconsistency by using a
different style just for this one.
Your patch looks good.