[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1099218 - /subversion/trunk/subversion/libsvn_wc/info.c

From: Greg Stein <gstein_at_gmail.com>
Date: Tue, 3 May 2011 16:25:58 -0400

On Tue, May 3, 2011 at 16:03, <hwright_at_apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_wc/info.c Tue May  3 20:03:13 2011
> @@ -172,11 +172,12 @@ build_info_for_entry(svn_info2_t **info,
>   /* A default */
>   tmpinfo->size = SVN_INVALID_FILESIZE;
>
> -  SVN_ERR(svn_wc__node_get_schedule(&tmpinfo->wc_info->schedule, NULL,
> -                                    wc_ctx, local_abspath, scratch_pool));
> +  SVN_ERR(svn_wc__internal_node_get_schedule(&tmpinfo->wc_info->schedule, NULL,
> +                                             wc_ctx->db, local_abspath,
> +                                             scratch_pool));

Why is wc_ctx being passed around these module-private functions?
Shouldn't you pop that open in the semi-public function, and pass
around DB?

Hm. Maybe because there are still references to svn_wc__node_* functions?

>...

Cheers,
-g
Received on 2011-05-03 22:26:26 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.