Johan Corveleyn wrote on Sun, Jan 30, 2011 at 15:32:00 +0100: > On Sun, Jan 30, 2011 at 4:17 AM, <danie...@apache.org> wrote: > > @@ -377,7 +381,6 @@ find_identical_prefix(svn_boolean_t *rea > > - apr_ssize_t max_delta, delta; ... > > @@ -503,7 +506,7 @@ find_identical_suffix(struct file_info f > > - svn_boolean_t is_match, can_read, can_read_word, reached_prefix; > > + svn_boolean_t is_match, can_read, reached_prefix; > > Same here. The variable 'can_read_word' is used inside the > SVN_UNALIGNED_ACCESS_IS_OK block, at line 573 and following. > > So, how should these be handled? Can we only declare those two > variables if SVN_UNALIGNED_ACCESS_IS_OK?
+1, sorry for the bug. (Usually I grep for a variable name when I get an "unused" error, I can't explain why I missed it this time.) Fixed in r1065305. > Or maybe we can rearrange the code a little bit to make it more clear > at the same time ... Yes; see the log message.
