You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2018/07/20 16:06:11 UTC

svn commit: r1836368 - /subversion/trunk/subversion/libsvn_client/repos_diff.c

Author: julianfoad
Date: Fri Jul 20 16:06:11 2018
New Revision: 1836368

URL: http://svn.apache.org/viewvc?rev=1836368&view=rev
Log:
Fix libsvn_client 'repos-diff' which was incompletely filtering out no-op
property changes since r1097039.
Tested by temporarily inserting postcondition assertions.

* subversion/libsvn_client/repos_diff.c
  (remove_non_prop_changes): Remove all redundant prop changes, not just some of them.

Modified:
    subversion/trunk/subversion/libsvn_client/repos_diff.c

Modified: subversion/trunk/subversion/libsvn_client/repos_diff.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/repos_diff.c?rev=1836368&r1=1836367&r2=1836368&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/repos_diff.c (original)
+++ subversion/trunk/subversion/libsvn_client/repos_diff.c Fri Jul 20 16:06:11 2018
@@ -413,6 +413,7 @@ remove_non_prop_changes(apr_hash_t *pris
                        = APR_ARRAY_IDX(changes, j+1, svn_prop_t);
                 }
               changes->nelts--;
+              i--;
             }
         }
     }