You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by hw...@apache.org on 2011/09/14 22:33:34 UTC

svn commit: r1170838 - /subversion/trunk/subversion/include/svn_editor.h

Author: hwright
Date: Wed Sep 14 20:33:34 2011
New Revision: 1170838

URL: http://svn.apache.org/viewvc?rev=1170838&view=rev
Log:
* subversion/include/svn_editor.h
  (svn_editor_set_props): Update docstring in answer to a question, and remove
    said question.

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=1170838&r1=1170837&r2=1170838&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_editor.h (original)
+++ subversion/trunk/subversion/include/svn_editor.h Wed Sep 14 20:33:34 2011
@@ -771,13 +771,8 @@ svn_editor_add_absent(svn_editor_t *edit
 
 /** Drive @a editor's #svn_editor_cb_set_props_t callback.
  *
- * Set or change properties on the existing node at @a relpath.
- * ### TODO @todo Does this send *all* properties, always?
- * ### 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.
- * ###
+ * Set or change properties on the existing node at @a relpath.  This
+ * function sends *all* properties, both existing and changes.
  * ### TODO @todo What is REVISION for?
  * ### HKW: This is puzzling to me as well...
  * ###