From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 21 Jun 2011 20:05:14 GMT
Subject: Re: [PATCH v2 16/20] transport-helper: update ref status after push with export
Message-ID: <7vpqm76jd1.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1308496725-22329-17-git-send-email-srabbelier@gmail.com>

Sverre Rabbelier <srabbelier@gmail.com> writes:

> +        changed = {}
> +
> +        for name, value in refs_after.iteritems():
> +            if refs_before.get(name) == value:
> +                continue
> +
> +            changed[name] = value

This assumes that nobody will ever _remove_ refs. I think it is a sensible
assumption but somebody might want to give a power to the fast-import stream
to say "I do not want that ref anymore".

I dunno; it is not a big deal to me either way.  Just something to think
about.