[PATCH 11/13] for_each_reflog(): don't abort for bad references
- From
Michael Haggerty <mhagger@alum.mit.edu>
- Date
- May 30, 2016, 07:55 UTC
- Message-ID
- <6228ed402dbdf9f937cb504aff6bbf8849771c54.1464537050.git.mhagger@alum.mit.edu>
- In-Reply-To
- <cover.1464537050.git.mhagger@alum.mit.edu>
If there is a file under "$GIT_DIR/logs" with no corresponding reference, the old code was emitting an error message, aborting the reflog iteration, and returning -1. But
* None of the callers was checking the exit value
* The callers all want to find all legitimate reflogs (sometimes for the purpose of determining object reachability!) and wouldn't benefit from a truncated iteration anyway.
So instead, emit an error message and skip the "broken" reflog, but continue with the iteration.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> --- refs/files-backend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/refs/files-backend.c b/refs/files-backend.c index c48a006..a7cc0e2 100644 --- a/refs/files-backend.c +++ b/refs/files-backend.c @@ -3325,7 +3325,7 @@ static int do_for_each_reflog(struct strbuf *name, each_ref_fn fn, void *cb_data struct object_id oid; if (read_ref_full(name->buf, 0, oid.hash, NULL)) - retval = error("bad ref for %s", name->buf); + error("bad ref for %s", name->buf); else retval = fn(name->buf, &oid, 0, cb_data); }
-- 2.8.1