From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 13 Aug 2014 21:18:56 GMT
Subject: Re: [PATCH v2 00/23] backend-struct-db
Message-ID: <xmqqmwb82im7.fsf@gitster.dls.corp.google.com>
In-Reply-To: <1407960907-18189-1-git-send-email-sahlberg@google.com>

Ronnie Sahlberg <sahlberg@google.com> writes:

> 15 of the patches, the refs-common.c patches, focuses on moving all backend
> agnostic refs functions to a common file. This file will contain all
> backend agnostic refs functions.
>
> The last 6 patches adds a backend structure with the methods we need to
> describe a pluggable backend. Currently we only have one built in backend,
> the current files based backend. These patches do not change any of the
> behavior other than that we now call the methods through backend specific
> wrapper functions rather than calling them directly.
>
> At this stage we now have a defined set of methods needed for a refs
> backend and we can start building and adding new types of ref backends
> to git.

Very nice ;-).

I would have expected that refs.c would be the generic one and refs-be-*
would be the backend specific ones, though; that way you do not have to
introduce a new file refs-common.c at all, no?