On 17.02.2011 12:47, Stefan Küng wrote: > Hi, > > The new function svn_wc__prop_list_recursive() provides a big > performance gain for listing all properties of a full working copy. > Currently I have to use svn_client_proplist3() to make use of that. > > But I'd like to have that API accessible directly, the same way e.g. > svn_wc_prop_list2(). It is much easier to use those APIs for accessing > the working copy because then I don't have to set up a full client > context. > > Of course if you like to keep that API private, I'm ok with that too > since I can use the svn_client_ API instead if I have to. > > Thoughts?
I think svn_wc_prop_list2 should be rev'd to *3 and given a depth parameter, and call proplist_recursive, because the latter can handle depth=empty, too. It should be possible to reimplement other public and not-so-public prop-reading functions as wrappers for the _recursive case. -- Brane
