From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 15 Aug 2016 21:05:26 GMT
Subject: Re: [PATCH 2/2] checkout: do not mention detach advice for explicit --detach option
Message-ID: <xmqqh9alg2uh.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20160815184730.eevqogqrxp2zp43q@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

> I don't think doing it this way is _wrong_. It just feels sort of
> pointlessly over-engineered. It's also a little weird that all of the:
>
>   if (advice_foo)
>
> will trigger because "advice_foo" is set to -1. I think it does the
> right thing, but it feels like a bug (the value is now a tri-state, and
> we silently collapse two states into one).

Guilty as charged.  I do agree that this is over-engineered.