[PATCH 0/3] a small branch API clean-up · Junio C Hamano <hidden> · 2017-10-13
[PATCH 2/3] branch: split validate_new_branchname() into two · Junio C Hamano <hidden> · 2017-10-13
Re: [PATCH 2/3] branch: split validate_new_branchname() into two · Kaartic Sivaraam <hidden> · 2017-10-21
[PATCH 1/3] branch: streamline "attr_only" handling in validate_new_branchname() · Junio C Hamano <hidden> · 2017-10-13
Re: [PATCH 1/3] branch: streamline "attr_only" handling in validate_new_branchname() · Eric Sunshine <hidden> · 2017-10-13
[PATCH 3/3] branch: forbid refs/heads/HEAD · Junio C Hamano <hidden> · 2017-10-13
Re: [PATCH 3/3] branch: forbid refs/heads/HEAD · Jeff King <hidden> · 2017-10-13
Re: [PATCH 0/3] a small branch API clean-up · Kaartic Sivaraam <hidden> · 2017-10-21
[PATCH v2 1/2] branch: forbid refs/heads/HEAD · Kaartic Sivaraam <hidden> · 2017-11-14
[PATCH v2 2/2] builtin/branch: remove redundant check for HEAD · Kaartic Sivaraam <hidden> · 2017-11-14
Re: [PATCH v2 1/2] branch: forbid refs/heads/HEAD · Kaartic Sivaraam <hidden> · 2017-11-14

Re: [PATCH 1/3] branch: streamline "attr_only" handling in validate_new_branchname()

From: Eric Sunshine <hidden>
Date: 2017-10-13 07:06:04

On Fri, Oct 13, 2017 at 1:11 AM, Junio C Hamano [off-list ref] 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 <redacted>

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help