From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 09 Feb 2015 22:26:13 GMT
Subject: Re: [PATCH 11/24] untracked cache: load from UNTR index extension
Message-ID: <xmqqmw4mwwl6.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1423385748-19825-12-git-send-email-pclouds@gmail.com>

Nguyễn Thái Ngọc Duy  <pclouds@gmail.com> writes:

> +static int read_one_dir(struct untracked_cache_dir **untracked_,
> +			struct read_data *rd)
> +{
> +#define NEXT(x) \
> +	next = data + (x); \
> +	if (next > rd->end) \
> +		return -1;

Please do not do this, and instead write them at two places they are
used.  It is not clear to those who are reading the use sites that
this can break the control flow and return to the caller.