From: Keller, Jacob E <jacob.e.keller@intel.com>
Date: Tue, 15 Jul 2014 20:29:12 GMT
Subject: Re: [PATCH 3/3] tag: support configuring --sort via .gitconfig
Message-ID: <1405456151.2577.7.camel@jekeller-desk1.amr.corp.intel.com>
In-Reply-To: <xmqqha2iv3kr.fsf@gitster.dls.corp.google.com>

On Tue, 2014-07-15 at 12:12 -0700, Junio C Hamano wrote:
> "Keller, Jacob E" <jacob.e.keller@intel.com> writes:
> 
> > I am going to re-submit this with an enum-style return. I am also
> > changing how we parse so that we can correctly report whether the sort
> > function or sort atom is incorrect.
> 
> Oh, our mails crossed, I guess.  As long as it will leave the door
> open for later enhancements for more context sensitive error
> diagnosis, I do not particularly mind a solution around enum.

Hmm. I looked at coding it this way, and it actually makes it less
sensitive than I would like. I'm not a fan of the extra "value"
parameter, but I do like a more proper error display, and indeed one
that is more precise.

I'll try to have a new series posted soon which takes these into
account.

Regards,
Jake