From: David Turner <dturner@twopensource.com>
Date: Wed, 24 Feb 2016 19:28:16 GMT
Subject: Re: [PATCH] refs: reduce the visibility of do_for_each_ref()
Message-ID: <1456342096.18017.2.camel@twopensource.com>
In-Reply-To: <56CBAA31.7030809@ramsayjones.plus.com>

Got it, thanks.

On Tue, 2016-02-23 at 00:39 +0000, Ramsay Jones wrote:
> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> ---
> 
> Hi David,
> 
> Again, If you need to re-roll your 'dt/refs-backend-lmdb' branch ...
> 
> Thanks!
> 
> ATB,
> Ramsay Jones
> 
>  refs.c               | 16 ++++++++--------
>  refs/refs-internal.h |  6 ------
>  2 files changed, 8 insertions(+), 14 deletions(-)
> 
> diff --git a/refs.c b/refs.c
> index 337f110..bf70876 100644
> --- a/refs.c
> +++ b/refs.c
> @@ -1236,6 +1236,14 @@ static int get_affected_refnames(struct
> ref_transaction *transaction,
>  	return 0;
>  }
>  
> +static int do_for_each_ref(const char *submodule, const char *base,
> +			   each_ref_fn fn, int trim, int flags,
> +			   void *cb_data)
> +{
> +	return the_refs_backend->do_for_each_ref(submodule, base,
> fn, trim,
> +						 flags, cb_data);
> +}
> +
>  int for_each_ref(each_ref_fn fn, void *cb_data)
>  {
>  	return do_for_each_ref(NULL, "", fn, 0, 0, cb_data);
> @@ -1501,14 +1509,6 @@ int resolve_gitlink_ref(const char *path,
> const char *refname,
>  	return the_refs_backend->resolve_gitlink_ref(path, refname,
> sha1);
>  }
>  
> -int do_for_each_ref(const char *submodule, const char *base,
> -		    each_ref_fn fn, int trim, int flags,
> -		    void *cb_data)
> -{
> -	return the_refs_backend->do_for_each_ref(submodule, base,
> fn, trim,
> -						 flags, cb_data);
> -}
> -
>  int for_each_reflog_ent_reverse(const char *refname,
> each_reflog_ent_fn fn,
>  				void *cb_data)
>  {
> diff --git a/refs/refs-internal.h b/refs/refs-internal.h
> index ced6af4..4922e25 100644
> --- a/refs/refs-internal.h
> +++ b/refs/refs-internal.h
> @@ -227,12 +227,6 @@ int do_for_each_per_worktree_ref(const char
> *submodule, const char *base,
>  				 each_ref_fn fn, int trim, int
> flags,
>  				 void *cb_data);
>  
> -/*
> - * The common backend for the for_each_*ref* functions
> - */
> -int do_for_each_ref(const char *submodule, const char *base,
> -		    each_ref_fn fn, int trim, int flags, void
> *cb_data);
> -
>  /* refs backends */
>  typedef int ref_init_db_fn(int shared, struct strbuf *err);
>  typedef int ref_transaction_commit_fn(struct ref_transaction
> *transaction,