From: Eric Sunshine <sunshine@sunshineco.com>
Date: Fri, 13 Oct 2017 07:05:59 GMT
Subject: Re: [PATCH 1/3] branch: streamline "attr_only" handling in validate_new_branchname()
Message-ID: <CAPig+cSey2M4jUE-VbdqyQjDWZL1PHfvei5VL1VE+KDxd82v6g@mail.gmail.com>
In-Reply-To: <20171013051132.3973-2-gitster@pobox.com>

On Fri, Oct 13, 2017 at 1:11 AM, Junio C Hamano <gitster@pobox.com> wrote:
> The function takes a parameter "attr_only" (which is a name that is
> hard to reason about, which will be corrected soon) and skips some
> checks when it is set.  Reorganize the conditionals to make it move

s/move/more/

> obvious that some checks are never made when this parameter is set.
>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>