Howdy folks,
    The current wording of svn_delta_editor_t's absent_(file|directory)
    callback functions is a bit misleading:
        "In the (file|directory) ... but cannot be conveyed to the
         consumer (perhaps because of authorization restrictions)."
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    Because of that wording, I just spent a chunk of time looking through
    the code trying to find if these callbacks would be invoked for any
    reason *other* than an authorization restriction.
    I've concluded that no, they wouldn't :-)
    (The only code path that triggers delta.c:absent_file_or_dir() is
     wrapped up in an authz test in delta.c:add_file_or_dir().)
    Attached patch improves the wording so that there's no ambiguity
    as to when absent_(file|directory) callbacks would be invoked.
    Regards,
        Trent.
[[[
Improve docstrings for svn_delta_editor_t's absent_(file|directory) callbacks.
* subversion/include/svn_delta.h
  (svn_delta_editor_t): the absent_(file|directory)
   callbacks will *only* be invoked due to authz
   restrictions; state this clearly in the docstrings.
Patch by: Trent Nelson <trent_at_snakebite.org>
]]]
Index: subversion/include/svn_delta.h
===================================================================
--- subversion/include/svn_delta.h	(revision 1229889)
+++ subversion/include/svn_delta.h	(working copy)
@@ -916,8 +916,8 @@
   /** In the directory represented by @a parent_baton, indicate that
    * @a path is present as a subdirectory in the edit source, but
-   * cannot be conveyed to the edit consumer (perhaps because of
-   * authorization restrictions).
+   * cannot be conveyed to the edit consumer because of authorization
+   * restrictions.
    *
    * Any temporary allocations may be performed in @a scratch_pool.
    */
@@ -1040,8 +1040,8 @@
   /** In the directory represented by @a parent_baton, indicate that
    * @a path is present as a file in the edit source, but cannot be
-   * conveyed to the edit consumer (perhaps because of authorization
-   * restrictions).
+   * conveyed to the edit consumer because of authorization
+   * restrictions.
    *
    * Any temporary allocations may be performed in @a scratch_pool.
    */
Received on 2012-01-11 05:57:24 CET