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 2012/07/10 18:16:43 UTC
svn commit: r1359753 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
Author: stefan2
Date: Tue Jul 10 16:16:43 2012
New Revision: 1359753
URL: http://svn.apache.org/viewvc?rev=1359753&view=rev
Log:
Follow-up to r1359574.
* subversion/libsvn_fs_fs/fs_fs.c
(write_packed_revprop): remove ";" within statement, don't post-increment /
-decrement macro parameters
Found by: philip
Modified:
subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
Modified: subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c?rev=1359753&r1=1359752&r2=1359753&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c Tue Jul 10 16:16:43 2012
@@ -4011,11 +4011,17 @@ write_packed_revprop(const char **final_
while (left <= right)
if ( left_size + APR_ARRAY_IDX(revprops->sizes, left, apr_off_t)
< right_size + APR_ARRAY_IDX(revprops->sizes, right, apr_off_t))
- left_size += APR_ARRAY_IDX(revprops->sizes, left++, apr_off_t)
- + SVN_INT64_BUFFER_SIZE;
- else
- right_size += APR_ARRAY_IDX(revprops->sizes, right--, apr_off_t);
+ {
+ left_size += APR_ARRAY_IDX(revprops->sizes, left, apr_off_t)
+ SVN_INT64_BUFFER_SIZE;
+ ++left;
+ }
+ else
+ {
+ right_size += APR_ARRAY_IDX(revprops->sizes, right, apr_off_t)
+ + SVN_INT64_BUFFER_SIZE;
+ ++right;
+ }
/* since the items need much less than SVN_INT64_BUFFER_SIZE
* bytes to represent their length, the split may not be optimal */