From: Clemens Buchacher Date: Mon, 11 Jul 2011 17:57:09 GMT Subject: Re: [PATCH 3/5] commit: add commit_generation function Message-ID: <20110711175709.GA6245@toss.lan> In-Reply-To: <20110711161814.GC10418@sigill.intra.peff.net> On Mon, Jul 11, 2011 at 12:18:14PM -0400, Jeff King wrote: > > +unsigned long commit_generation(const struct commit *commit) > +{ [...] > + /* drop const because we may call parse_commit */ > + return commit_generation_recurse((struct commit *)commit); > +} Out of curiosity, why make it const in the first place? Clemens