From: Jon Seymour Date: Thu, 08 Sep 2011 01:23:25 GMT Subject: Re: [PATCH v17 1/7] bisect: move argument parsing before state modification. Message-ID: In-Reply-To: <4E67B2F2.9070806@kdbg.org> On Thu, Sep 8, 2011 at 4:07 AM, Johannes Sixt wrote: > Am 07.09.2011 08:16, schrieb Christian Couder: > IOW, I think the new behavior is *much* better than the old behavior. > There is perhaps no surprise that I agree with Hannes. Certainly, it seemed saner to me to do argument validation before state update. [ Also, the earlier iterations of the --no-checkout series needed the new behaviour. Not sure if that is still true, but I suspect it is ]. jon.