Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> 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.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html