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);