Author: rhuijben
Date: Mon Mar 17 11:44:22 2014
New Revision: 1578316
URL: http://svn.apache.org/r1578316
Log:
* STATUS: Extend r1578273 group
Modified:
subversion/branches/1.8.x/STATUS
Modified: subversion/branches/1.8.x/STATUS
URL:
http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1578316&r1=1578315&r2=1578316&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Mon Mar 17 11:44:22 2014
@@ -270,13 +270,19 @@ Candidate changes:
Votes:
+1: rhuijben
- * r1578273
- Stop svn_ra_get_locks() on ra_serf reporting non locks as svn_lock_t.
+ * r1578273, r1578311
+ Make svn_ra_get_locks() and svn_ra_get_lock() report not locked nodes
+ with a NULL svn_lock_t *, as documented.
Justification:
Many clients use the existance of an svn_lock_t on a node via info/status
as a boolean to note that there is a lock. But because we didn't properly
- check the result we reported svn_lock_t instances in more cases: even on
- directories.
+ check the result we reported mostly empty svn_lock_t instances in more
+ cases. Even on directories!
+ Notes:
+ A branch is needed, because the serf lock handling code was split to
+ multiple files on trunk.
+ Branch:
+ ^/subversion/branches/1.8.x-r1578311
Votes:
+1: rhuijben
