]> granicus.if.org Git - git/commitdiff
bundle: Fix "verify" output if history is complete
authorLukas Fleischer <git@cryptocrack.de>
Thu, 7 Mar 2013 00:56:35 +0000 (01:56 +0100)
committerJunio C Hamano <gitster@pobox.com>
Thu, 7 Mar 2013 21:33:56 +0000 (13:33 -0800)
A more informative message for "complete" bundles was added in commit
8c3710fd3011 (tweak "bundle verify" of a complete history, 2012-06-04).

However, the prerequisites ref list is currently read *after* we
check if it equals zero, which means we never actually use the
number of prerequisite refs to decide when to print the newly
introduced message.  The code incorrectly uses the number of
references recorded in the bundle instead.

Signed-off-by: Lukas Fleischer <git@cryptocrack.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
bundle.c

index 8d12816b9d0bc682ed9c019a7a5d5cec4b859171..65db53b6cb11f8dffa07d261085ee39a17da3ddc 100644 (file)
--- a/bundle.c
+++ b/bundle.c
@@ -188,10 +188,10 @@ int verify_bundle(struct bundle_header *header, int verbose)
                             r->nr),
                          r->nr);
                list_refs(r, 0, NULL);
+               r = &header->prerequisites;
                if (!r->nr) {
                        printf_ln(_("The bundle records a complete history."));
                } else {
-                       r = &header->prerequisites;
                        printf_ln(Q_("The bundle requires this ref",
                                     "The bundle requires these %d refs",
                                     r->nr),