[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r17211 - in branches/wc-propcaching/subversion: libsvn_wc

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-11-14 13:50:54 CET

On Sat, 12 Nov 2005, Ivan Zhakov wrote:

> On 11/6/05, lundblad@tigris.org <lundblad@tigris.org> wrote:
> Author: lundblad
> Date: Sat Nov 5 18:44:38 2005
> New Revision: 17211
>
> Modified:
> branches/wc-propcaching/subversion/include/svn_wc.h
> branches/wc-propcaching/subversion/libsvn_wc/adm_ops.c
> branches/wc-propcaching/subversion/libsvn_wc/entries.c
> branches/wc-propcaching/subversion/libsvn_wc/entries.h
> branches/wc-propcaching/subversion/libsvn_wc/log.c
> branches/wc-propcaching/subversion/libsvn_wc/props.c
> branches/wc-propcaching/subversion/libsvn_wc/props.h
>
> Log:
> On wc-propcaching branch: Add a flag to the entries file keeping track of
> whether properties were modified or not. Keep this flag updated and
> use it to determined whether properties are modified. Add lots of TODO
> markers in the code.
>
> NOTE: This commit breaks stat_tests 13, which tests thats certain properties
> (pun partly intended;) of property timestamps. This has to be resolved.
>
> * subversion/include/svn_wc.h (svn_wc_entry_t): Add prop_mods member.
>
> * subversion/libsvn_wc/props.c
> (svn_wc__install_props): New function. Parts factored from
> svn_wc__merge_props.
> (svn_wc__merge_props, svn_wc_prop_set2): Use svn_wc__isntall_props.
>Peter, why you didn't use svn_wc__install_props() in
>svn_wc__merge_prop_diffs(). This should simplify and reduce code.

I've added a TODO to revisit this before merging to trunk.

>Because it deprecated and not tested or you missed it?

Don't ask:-)

Thanks for reviewing,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Nov 14 13:52:51 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.