Hi Stefan.

Just a quick note to say thanks for this, but there seems to be a
problem with this version when compiling:

  text_delta.c:709: warning: comparison between pointer and integer

and when running the test suite in a trunk head build:

  text_delta.c:693: svn_txdelta_apply_instructions: Assertion `left > 0'
failed.

- Julian


On Sat, 2010-05-22, Stefan Fuhrmann wrote:
> Hi there,
> 
> this is an improved version of the patch posted here:
> 
>   
> http://mail-archives.apache.org/mod_mbox/subversion-dev/201005.mbox/%3c4bf5bc07.8060...@alice-dsl.de%3e
> 
> It should address all the questions raised in that thread by
> 
> * keeping it as close to the original as possible
>   by looping over window ops instead of the target buffer
> * disabling the chunky copy optimization on non-x86/x64
> * vastly extended commentary explaining the consistency
>   requirements as well as the role of the assertions.
> 
> -- Stefan^2.


Reply via email to