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

Posted to commits@subversion.apache.org by rh...@apache.org on 2015/08/24 17:06:17 UTC

svn commit: r1697429 - /subversion/branches/1.9.x/STATUS

Author: rhuijben
Date: Mon Aug 24 15:06:17 2015
New Revision: 1697429

URL: http://svn.apache.org/r1697429
Log:
* STATUS: Cast some votes. Move swig fix to approved.

Modified:
    subversion/branches/1.9.x/STATUS

Modified: subversion/branches/1.9.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1697429&r1=1697428&r2=1697429&view=diff
==============================================================================
--- subversion/branches/1.9.x/STATUS (original)
+++ subversion/branches/1.9.x/STATUS Mon Aug 24 15:06:17 2015
@@ -74,15 +74,7 @@ Candidate changes:
      RA_svn is basically unusuable on that platform.
    Votes:
      +1: stefan2
-
- * r1694929
-   Fix memory corruption in SWIG bindings' svn_client_copy_source_t
-   constructor.
-   Justification:
-     Constructor result is not correct as REVISION gets partially overwritten
-     by PEG_REVISION. May also result in segfaults.
-   Votes:
-     +1: stefan2, philip
+     +1: rhuijben (Would prefer clarity over the tiny performance improvement)
 
  * r1695022
    Fix incomplete membuffer cache initialization.
@@ -143,3 +135,13 @@ Approved changes:
      This is a regression as older Subversion releases ignored the error.
    Votes:
      +1: rhuijben, brane, stefan2
+
+ * r1694929
+   Fix memory corruption in SWIG bindings' svn_client_copy_source_t
+   constructor.
+   Justification:
+     Constructor result is not correct as REVISION gets partially overwritten
+     by PEG_REVISION. May also result in segfaults.
+   Votes:
+     +1: stefan2, philip
+     +0.5: rhuijben (Minimal review)