On phone. Sorry for terse. On Sep 13, 2011 9:54 AM, <[email protected]> wrote: > > Author: hwright > Date: Tue Sep 13 19:54:14 2011 > New Revision: 1170324 > > URL: http://svn.apache.org/viewvc?rev=1170324&view=rev > Log: > * subversion/include/svn_editor.h > (svn_editor_set_props): Add a couple of notes. > > Modified: > subversion/trunk/subversion/include/svn_editor.h > > Modified: subversion/trunk/subversion/include/svn_editor.h > URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_editor.h?rev=1170324&r1=1170323&r2=1170324&view=diff > ============================================================================== > --- subversion/trunk/subversion/include/svn_editor.h (original) > +++ subversion/trunk/subversion/include/svn_editor.h Tue Sep 13 19:54:14 2011 > @@ -773,7 +773,14 @@ svn_editor_add_absent(svn_editor_t *edit > * > * Set or change properties on the existing node at @a relpath. > * ### TODO @todo Does this send *all* properties, always?
Yes. > + * ### HKW: The purist in me says "yes", but the pragmatist says "no". > + * ### Writing a backward compat shim is going to be next to impossible, > + * ### since we've no way to know about existing props while receiving > + * ### props from the delta editor. An incoming propdelta always knows the target to apply against. Your shim must fetch the target props, compute the delta, and call the right callbacks. > + * ### > * ### TODO @todo What is REVISION for? > + * ### HKW: This is puzzling to me as well... What rev are you attempting to set props against? Are you out of date? > + * ### > * ### what about "entry props"? will these still be handled via > * ### the general prop function? > * > >
