On Thu, Jun 20, 2013 at 08:13:14PM +0400, Ivan Zhakov wrote: > On Thu, Jun 20, 2013 at 8:02 PM, Mattias Engdegård > <matti...@bredband.net> wrote: > > 20 jun 2013 kl. 16.54 skrev Ivan Zhakov: > > > > > >> It seems the code is missing #ifdef SVN_UNALIGNED_ACCESS_IS_OK . The > >> attached patch should fix problem, but I'm not sure that this is right > >> solution for the problem. > > > > > > Assuming the hash function does not need to be portable (which appears > > likely from a cursory look at the code), that fix is basically correct, if > > you move out the assignment i=0 to right before the section you made > > conditional. The following loop will complete the hash. > > In other words, something like: > > > Yes, I missed "i = 0" assignment. Thanks for fixing. > > Stefan, I think Mattias is faster and prefer this approach actually.
Sure. I just committed my fix because it helps, but won't stand in the way of other fixes. I myself would let Stefan Fuhrmann make a final decision on this.