From: Junio C Hamano Date: Sun, 22 Jul 2007 09:17:47 GMT Subject: Re: [PATCH] Avoid to duplicate commit message when is not encoded Message-ID: <7v4pjwssp0.fsf@assigned-by-dhcp.cox.net> In-Reply-To: <20070722085225.GB3408@steel.home> Alex Riesen writes: > Marco Costalba, Sun, Jul 22, 2007 10:23:05 +0200: >> + if (encoding) // we'll strip encoding header later > > You better avoid using C++ comments, not all C compilers support them. I've "fixed" them. >> + else >> + return NULL; // nothing to do > > And you better avoid such useless comments too I think this one is probably a good-to-have. It really needs a bit of thinking why there is nothing to do in this case. At least for me it did ;-)