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 2012/02/04 17:56:04 UTC

svn commit: r1240546 - /subversion/trunk/subversion/libsvn_delta/compat.c

Author: hwright
Date: Sat Feb  4 16:56:03 2012
New Revision: 1240546

URL: http://svn.apache.org/viewvc?rev=1240546&view=rev
Log:
Ev2 shims: Remove a superfluous call to the fetch_kind callback.

* subversion/libsvn_delta/compat.c
  (build): Don't call the fetch_kind_func() when setting properties, as we
    already know what the kind will be.

Modified:
    subversion/trunk/subversion/libsvn_delta/compat.c

Modified: subversion/trunk/subversion/libsvn_delta/compat.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_delta/compat.c?rev=1240546&r1=1240545&r2=1240546&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_delta/compat.c (original)
+++ subversion/trunk/subversion/libsvn_delta/compat.c Sat Feb  4 16:56:03 2012
@@ -1071,16 +1071,7 @@ build(struct editor_baton *eb,
       apr_hash_t *current_props;
       apr_array_header_t *propdiffs;
 
-      /* Only fetch the kind if operating on something other than the root,
-         as we already know the kind on the root. */
-      if (operation->path)
-        {
-          if (kind == svn_kind_unknown)
-            SVN_ERR(eb->fetch_kind_func(&operation->kind, eb->fetch_kind_baton,
-                                        relpath, rev, scratch_pool));
-          else
-            operation->kind = kind;
-        }
+      operation->kind = kind;
 
       if (operation->operation == OP_REPLACE)
         current_props = apr_hash_make(scratch_pool);