Kazutoshi Satoda <k_satoda@f2.dion.ne.jp> writes:
> Without the extra request, we can see that the server does not support
> deadprop-count only after fetching properties for all dirent_fields except
> SVN_DIRENT_HAS_PROPS. In the legacy case, we need extra depth 1 allprop
> request for SVN_DIRENT_HAS_PROPS. I think that such request will be very
> redundant because there is no way to request allprops except we have
> already fetched.
>
> The extra request tests a server capability, not an actual value to fill
> some dirent field. It might be done by checking the server version, but
> I don't know whether it is possible or not.
>
> I thought to combine the extra request with the last 'if(props)' part.
> But I didn't do that to minimize the change to the previous patch.
Thanks for the explanation. We still have a speedup, obviously, so
I'm not too worried, I just wanted to know if even greater efficiency
was possible.
When you "the last 'if(props)' part", do you mean the ones at the ends
of svn_ra_dav__get_file() and svn_ra_dav__get_dir()?
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Mar 7 00:56:06 2006