Re: [wc-propcaching]: Cached proprety existence, take two

On 11/15/05, Peter N. Lundblad <peter@famlundblad.se> wrote:
> On Mon, 14 Nov 2005, Daniel Berlin wrote:
>
> > > No need to clear the entry first. That will just confuse readers.
> >
> > Actually, you do need to.
> >
> > It will live on from tmp_entry above.
> >
> So, you have a tmp_entry, you clear it and then assign to the only field
> that will be used by the following call. How can that be necessary?
>
> BTW, you can leave this svn_wc_merge_prop_diffs function alone for now.
> I've left a TODO at the top (after you created this patch). We should look
> into reimplementing this in terms of svn_wc__install_props.
No problem to implement svn_wc_merge_prop_diffs in terms of
svn_wc__install_props, but ideally we should implement it using
svn_wc__merge_props with server_baseprops NULL.

--
Ivan Zhakov