stef...@apache.org writes: > Author: stefan2 > Date: Tue Jul 10 10:19:42 2012 > New Revision: 1359574
> + else > + right_size += APR_ARRAY_IDX(revprops->sizes, right--, apr_off_t); > + + SVN_INT64_BUFFER_SIZE; ../src/subversion/libsvn_fs_fs/fs_fs.c: In function ‘write_packed_revprop’: ../src/subversion/libsvn_fs_fs/fs_fs.c:4018: warning: statement with no effect Also, putting right-- in a macro like APR_ARRAY_IDX relies on the macro not expanding to use the parameter twice. -- Philip