You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by st...@apache.org on 2013/06/26 13:59:52 UTC

svn commit: r1496891 - /subversion/branches/1.8.x/STATUS

Author: stefan2
Date: Wed Jun 26 11:59:51 2013
New Revision: 1496891

URL: http://svn.apache.org/r1496891
Log:
* STATUS: add r1496886 to 1494966 group and remove the -0s now addressed

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=1496891&r1=1496890&r2=1496891&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Wed Jun 26 11:59:51 2013
@@ -92,16 +92,15 @@ Candidate changes:
    Votes:
      +1: rhuijben, stsp
 
- * r1494966
+ * r1494966, r1496886
    Eliminate an extra RA layer roundtrip in 'svn log' on repository roots.
    Justification:
      Minor performance regression against 1.7.x plus requesting e.g. single
      revision logs on a repo root is not a fringe use-case.
+   Notes:
+     r1496886 properly handles the revision parameter edge-cases.
    Votes:
-     +1: stefan2, rhuijben
-     -0: stsp (looks like this code generates invalid location segments
-               for the root node, see dev@ r1494966 thread)
-     -0: ivan (doesn't handle SVN_INVALID_REVNUM)
+     +1: stefan2
 
  * ^/subversion/branches/1.8.x-r1495063
    r1495806, r1495985