From: Ævar Arnfjörð Bjarmason Date: Sat, 07 Aug 2010 17:39:38 GMT Subject: Re: [PATCH v2] Allow HTTP user agent string to be modified. Message-ID: In-Reply-To: <1281202184.4815.1.camel@compaq.olson.home> On Sat, Aug 7, 2010 at 17:29, Spencer E. Olson 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.