svn commit: r1587093 - /subversion/trunk/subversion/libsvn_ra_serf/commit.c

[rhuijben]

Author: rhuijben
Date: Sun Apr 13 20:10:02 2014
New Revision: 1587093

URL: http://svn.apache.org/r1587093
Log:
* subversion/libsvn_ra_serf/commit.c
  (create_proppatch_body): Don't create an unneeded
    '<D:set><D:prop></D:prop></D:set>' when committing file or directory
    property deletions, by making a bit of code conditional of the old value
    hashes.

Modified:
    subversion/trunk/subversion/libsvn_ra_serf/commit.c

Modified: subversion/trunk/subversion/libsvn_ra_serf/commit.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/commit.c?rev=1587093&r1=1587092&r2=1587093&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/commit.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/commit.c Sun Apr 13 20:10:02 2014
@@ -850,8 +850,11 @@ create_proppatch_body(serf_bucket_t **bk
       svn_ra_serf__add_close_tag_buckets(body_bkt, alloc, "D:set");
     }
 
-  if (apr_hash_count(ctx->removed_props) > 0)
+  if ((apr_hash_count(ctx->removed_props) > 0)
+      && (wb.previous_changed_props || wb.previous_removed_props))
     {
+      /* For revision properties we handle a remove as a special propset if
+         we want to provide the old version of the property */
       svn_ra_serf__add_open_tag_buckets(body_bkt, alloc, "D:set", SVN_VA_NULL);
       svn_ra_serf__add_open_tag_buckets(body_bkt, alloc, "D:prop", 
SVN_VA_NULL);