Currently a call to check_stages is using bit-shift values to define what will be used as the first argument. If the reader doesn't know what the values are from heart, they have to go dig in cache.h what each value is. This patch is replacing those values for the macros from cache.h so that it's clear at first sight what they are (CE_ENTRY_REMOVED, CE_ENTRY_ADDED) Signed-off-by: Edmundo Carmona Antoranz <eantoranz@xxxxxxxxx> --- builtin/checkout.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/checkout.c b/builtin/checkout.c index ffa776c6e1..3a644b31f6 100644 --- a/builtin/checkout.c +++ b/builtin/checkout.c @@ -397,7 +397,7 @@ static int checkout_paths(const struct checkout_opts *opts, } else if (opts->writeout_stage) { errs |= check_stage(opts->writeout_stage, ce, pos, opts->overlay_mode); } else if (opts->merge) { - errs |= check_stages((1<<2) | (1<<3), ce, pos); + errs |= check_stages(CE_ENTRY_REMOVED | CE_ENTRY_ADDED, ce, pos); } else { errs = 1; error(_("path '%s' is unmerged"), ce->name); -- 2.26.2