You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Barry Scott <ba...@barrys-emacs.org> on 2007/05/13 10:42:26 UTC

svn_list_proplist needs the depth API

I noticed this comment in svn_client.h on Trunk while investigating the
very welcome new depth parameter changes.

  * ### TODO(sd): I don't see any reason to change this recurse  
parameter
* ### to a depth right now; it's not exactly part of the
* ### sparse-directories feature, although it's related.  Usually
* ### you would just name the target carefully... Is there a
* ### situation where depth support would be useful here?
*
* If @a target is not found, return the error @c  
SVN_ERR_ENTRY_NOT_FOUND.
*
* @since New in 1.5.
*/
svn_error_t *
svn_client_proplist3(const char *target,
                      const svn_opt_revision_t *peg_revision,
                      const svn_opt_revision_t *revision,
                      svn_boolean_t recurse,
                      svn_proplist_receiver_t receiver,
                      void *receiver_baton,
                      svn_client_ctx_t *ctx,
                      apr_pool_t *pool);


One of the biggest performance problems with SVN API is the inability  
to quickly get all the
properties of a single directory. GUI programs need to get all the  
properties of the files
and subdirs in one fast operation.

In pysvn Workbench I had to by pass the SVN API. My code reads the  
prop_base
files because its unacceptable to freeze the GUI for many seconds to  
get trivia amounts
of data.

Please implement depth on the svn_client_prop* API functions.

Barry

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn_list_proplist needs the depth API

Posted by Barry Scott <ba...@barrys-emacs.org>.
On May 13, 2007, at 18:12, Vlad Georgescu wrote:

> Barry Scott wrote:
>> Please implement depth on the svn_client_prop* API functions.
>
> I added depth support to svn_client_proplist3 in r25007.
>

That's great. Thanks very much.

Barry


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn_list_proplist needs the depth API

Posted by Vlad Georgescu <vg...@gmail.com>.
Barry Scott wrote:
> Please implement depth on the svn_client_prop* API functions.

I added depth support to svn_client_proplist3 in r25007.

-- 
Vlad

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org