Karthik Nayak <karthik....@gmail.com> writes:
> --- a/ref-filter.c
> +++ b/ref-filter.c
> @@ -901,12 +903,19 @@ static int ref_filter_handler(const char *refname, 
> const struct object_id *oid,
>       if (!match_points_at(&filter->points_at, oid->hash, refname))
>               return 0;
>  
> +     if (filter->merge_commit) {
> +             commit = lookup_commit_reference_gently(oid->hash, 1);
> +             if (!commit)
> +                     return 0;
> +     }

I'd appreciate a comment here. If I understand correctly, the comment
could be along the lines of

/*
 * A merge filter implies that we're looking at refs pointing to
 * commits => discard non-commits early. The actual filtering is done
 * later.
 */

(perhaps something more concise)

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to