You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sv...@apache.org on 2013/05/25 06:01:46 UTC

svn commit: r1486287 - in /subversion/branches/1.8.x: ./ STATUS subversion/libsvn_repos/fs-wrap.c

Author: svn-role
Date: Sat May 25 04:01:45 2013
New Revision: 1486287

URL: http://svn.apache.org/r1486287
Log:
Merge r1483557 from trunk:

 * r1483557
   Match our revision property names exactly instead of by prefix only.
   Justification:
     While the impact of this bug is certainly low, it is still an easy
     to fix problem.
   Votes:
     +1: stefan2 (for 1.8.1), rhuijben, pburba

Modified:
    subversion/branches/1.8.x/   (props changed)
    subversion/branches/1.8.x/STATUS
    subversion/branches/1.8.x/subversion/libsvn_repos/fs-wrap.c

Propchange: subversion/branches/1.8.x/
------------------------------------------------------------------------------
  Merged /subversion/trunk:r1483557

Modified: subversion/branches/1.8.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1486287&r1=1486286&r2=1486287&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Sat May 25 04:01:45 2013
@@ -134,11 +134,3 @@ Approved changes:
 # that would restart the soak should not be added unless they are resolving
 # blocking issues.  If in doubt see this link for details:
 # http://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
-
- * r1483557
-   Match our revision property names exactly instead of by prefix only.
-   Justification:
-     While the impact of this bug is certainly low, it is still an easy
-     to fix problem.
-   Votes:
-     +1: stefan2 (for 1.8.1), rhuijben, pburba

Modified: subversion/branches/1.8.x/subversion/libsvn_repos/fs-wrap.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/subversion/libsvn_repos/fs-wrap.c?rev=1486287&r1=1486286&r2=1486287&view=diff
==============================================================================
--- subversion/branches/1.8.x/subversion/libsvn_repos/fs-wrap.c (original)
+++ subversion/branches/1.8.x/subversion/libsvn_repos/fs-wrap.c Sat May 25 04:01:45 2013
@@ -409,10 +409,8 @@ svn_repos_fs_revision_prop(svn_string_t 
   else if (readability == svn_repos_revision_access_partial)
     {
       /* Only svn:author and svn:date are fetchable. */
-      if ((strncmp(propname, SVN_PROP_REVISION_AUTHOR,
-                   sizeof(SVN_PROP_REVISION_AUTHOR)-1) != 0)
-          && (strncmp(propname, SVN_PROP_REVISION_DATE,
-                      sizeof(SVN_PROP_REVISION_DATE)-1) != 0))
+      if ((strcmp(propname, SVN_PROP_REVISION_AUTHOR) != 0)
+          && (strcmp(propname, SVN_PROP_REVISION_DATE) != 0))
         *value_p = NULL;
 
       else