On Sat, Aug 7, 2010 at 17:29, Spencer E. Olson <olsonse@xxxxxxxxx> 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. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html