From: Johannes Schindelin Date: Tue, 04 Jul 2006 10:50:50 GMT Subject: Re: [PATCH] Teach rev-parse the ... syntax. Message-ID: In-Reply-To: <873bdhbv4x.fsf@gmail.com> Hi, On Tue, 4 Jul 2006, Santi Béjar wrote: > + struct commit_list *exclude = get_merge_bases(a, b,1); You never free_commit_list() exclude. Side thought: we do not really support multiple ranges, do we? E.g. git-rev-list HEAD~10..HEAD~8 HEAD^.. would not yield the intended result, right? (And same goes for ... ranges) Maybe we should at least warn about that. Ciao, Dscho