From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mon, 04 May 2009 14:50:41 GMT
Subject: Re: [JGIT PATCH 1/6] Add set to IntList
Message-ID: <alpine.DEB.1.00.0905041649570.5553@intel-tinevez-2-302>
In-Reply-To: <20090504142231.GS23604@spearce.org>

Hi,

On Mon, 4 May 2009, Shawn O. Pearce wrote:

> Robin Rosenberg <robin.rosenberg.lists@dewire.com> wrote:
> > l?rdag 02 maj 2009 04:08:42 skrev "Shawn O. Pearce" <spearce@spearce.org>:
> > > Some applications may wish to modify an int list.
> 
> OK, as it turns out, this is used by code which Dscho wrote but I 
> haven't packaged up as patches yet.  And I don't think that even uses 
> the weird set(size(), ...) expands the list rules I implemented. So drop 
> this patch for now from the series and I'll respin it at a later date.

Indeed, my code only needs add(int i), which should expand the array on 
demand (i.e. after a clear(), it should _not_ blindly resize the array).

Ciao,
Dscho