You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by cm...@apache.org on 2013/02/14 15:34:11 UTC

svn commit: r1446204 - /subversion/trunk/subversion/include/private/svn_diff_tree.h

Author: cmpilato
Date: Thu Feb 14 14:34:11 2013
New Revision: 1446204

URL: http://svn.apache.org/r1446204
Log:
* subversion/include/private/svn_diff_tree.h
  Comment tweak.  Mostly indentation/whitespace stuff, but also remove
  one doubled word ('describe describe').

Modified:
    subversion/trunk/subversion/include/private/svn_diff_tree.h

Modified: subversion/trunk/subversion/include/private/svn_diff_tree.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/private/svn_diff_tree.h?rev=1446204&r1=1446203&r2=1446204&view=diff
==============================================================================
--- subversion/trunk/subversion/include/private/svn_diff_tree.h (original)
+++ subversion/trunk/subversion/include/private/svn_diff_tree.h Thu Feb 14 14:34:11 2013
@@ -37,7 +37,7 @@ extern "C" {
 /* 
  *                   About the diff tree processor.
  *
- * Subversion uses two kinds of editors to describe changes. One to describe
+ * Subversion uses two kinds of editors to describe changes. One to
  * describe changes on how to *exactly* transform one tree to another tree,
  * as efficiently as possible and one to describe the difference between trees
  * in order to review the changes, or to allow applying them on a third tree
@@ -61,43 +61,41 @@ extern "C" {
  * arguments to the already overloaded diff callbacks the api was completely
  * redesigned with a few points in mind.
  *
- *  * It must be able to drive the old callbacks interface without users
- *    noticing the difference (100% compatible).
- *    (Implemented as svn_wc__wrap_diff_callbacks())
+ *   * It must be able to drive the old callbacks interface without users
+ *     noticing the difference (100% compatible).
+ *     (Implemented as svn_wc__wrap_diff_callbacks())
  *
- *  * It should provide the information that was missing in the old interface,
- *    but required to close existing issues.
+ *   * It should provide the information that was missing in the old interface,
+ *     but required to close existing issues.
  *
  *     E.g. - properties and children on deleted directories. 
  *          - revision numbers and copyfrom information on directories.
  *
- *
  * To cleanup the implementation and make it easier on diff processors to
  * handle the results I also added the following constraints.
  *
- *  * Diffs should be fully reversable: anything that is deleted should be
- *    available, just like something that is added.
- *    (Proven via svn_diff__tree_processor_reverse_create)
- *    ### Still in doubt if *_deleted() needs a copy_to argument, for the
- *        99% -> 100%.
- *
- *  * Diff processors should have an easy way to communicate that they are
- *    not interrested in certain expensive to obtain results.
- *
- *  * Directories should have clear open and close events to allow adding them
- *    before their children, but still allowing property changes to have
- *    defined behavior.
+ *   * Diffs should be fully reversable: anything that is deleted should be
+ *     available, just like something that is added.
+ *     (Proven via svn_diff__tree_processor_reverse_create)
+ *     ### Still in doubt if *_deleted() needs a copy_to argument, for the
+ *     ### 99% -> 100%.
+ *
+ *   * Diff processors should have an easy way to communicate that they are
+ *     not interrested in certain expensive to obtain results.
+ *
+ *   * Directories should have clear open and close events to allow adding them
+ *     before their children, but still allowing property changes to have
+ *     defined behavior.
  *
- *  * Files and directories should be handled as similar as possible as in
- *    many cases they are just nodes in a tree.
+ *   * Files and directories should be handled as similar as possible as in
+ *     many cases they are just nodes in a tree.
  *
- *  * It should be easy to create diff wrappers to apply certain transforms.
+ *   * It should be easy to create diff wrappers to apply certain transforms.
  *
  * During the creation an additional requirement of knowing about 'some
  * absent' nodes was added, to allow the merge to work on just this processor
  * api.
  *
- *
  * The api describes a clean open-close walk through a tree, depending on the
  * driver multiple siblings can be described at the same time, but when a
  * directory is closed all descendants are done.