From: Jonathan Nieder <jrnieder@gmail.com>
Date: Fri, 13 May 2011 10:37:56 GMT
Subject: Re: [PATCH 0/8] Sequencer Foundations
Message-ID: <20110513103756.GC30618@elie>
In-Reply-To: <BANLkTi=8BrFXfoDwL_fXG2bXarP7d0xioA@mail.gmail.com>

Christian Couder wrote:

> About writing files before each cherry-pick, I am not against it, if
> it is really needed to be safe. I even suggested it in my patch series
> back in November
> (http://article.gmane.org/gmane.comp.version-control.git/162183).
> But it will make cherry-pick less efficient, so it is a kind of
> performance regression that we can perhaps avoid by changing some
> die() into error().

Yes, that's a good point.  Maybe in the long term the extra safety
could become optional.  And I am happy about the die() elimination;
the only part I was not as thrilled about is relying on it.

Some die() calls, like the one in xmalloc, would be very difficult to
eliminate.