[PATCH 1/4] pack_refs(): remove redundant check
- From
mhagger@alum.mit.edu <mhagger@alum.mit.edu>
- Date
- Jan 17, 2012, 05:50 UTC
- Message-ID
- <1326779434-20106-2-git-send-email-mhagger@alum.mit.edu>
- In-Reply-To
- <1326779434-20106-1-git-send-email-mhagger@alum.mit.edu>
From: Michael Haggerty <mhagger@alum.mit.edu>
handle_one_ref() only adds refs to the cbdata.ref_to_prune list if (cbdata.flags & PACK_REFS_PRUNE) is set. So any references in this list at the end of pack_refs() can be pruned unconditionally.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu> --- Trivial simplification, not essential to the rest of this patch series.
pack-refs.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/pack-refs.c b/pack-refs.c index 23bbd00..f09a054 100644 --- a/pack-refs.c +++ b/pack-refs.c @@ -143,7 +143,6 @@ int pack_refs(unsigned int flags) packed.fd = -1; if (commit_lock_file(&packed) < 0) die_errno("unable to overwrite old ref-pack file"); - if (cbdata.flags & PACK_REFS_PRUNE) - prune_refs(cbdata.ref_to_prune); + prune_refs(cbdata.ref_to_prune); return 0; }
-- 1.7.8.3