You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by "Hyrum K. Wright" <hy...@mail.utexas.edu> on 2010/05/14 22:39:23 UTC

Re: svn commit: r943722 - in /subversion/trunk/subversion: include/private/svn_wc_private.h libsvn_client/commit_util.c libsvn_wc/node.c

On Wed, May 12, 2010 at 5:21 PM, <ph...@apache.org> wrote:

> Author: philip
> Date: Wed May 12 22:21:57 2010
> New Revision: 943722
>
> URL: http://svn.apache.org/viewvc?rev=943722&view=rev
> Log:
> Remove svn_wc_entry_t from harvest_committables.
>
> * subversion/libsvn_clint/commit_util.c
>  (harvest_committables): Remove entry, parent_entry and copyfrom_url
>   parameters, add copy_root parameter, adjust recursive call, remove
>   use of svn_wc_entry_t.
>  (svn_client__harvest_committables): Adjust call to harvest_committables.
>  (harvest_copy_committables): Remove svn_wc_entry_t, adjust call to
>   harvest_committables.
>
> * subversion/include/private/svn_wc_private.h
>  (svn_wc__db_temp_get_keep_local): New.
>
> * subversion/libsvn_wc/node.c
>  (svn_wc__db_temp_get_keep_local): New.
>

Huh?  svn_wc__db_* functions are all internal to libsvn_wc.  I'd suggest
another name for this temporary function, if it does indeed need exposure
outside of libsvn_wc.

-Hyrum