[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r9280 - in trunk/subversion: include libsvn_delta

From: Branko Čibej <brane_at_xbc.nu>
Date: 2004-04-03 18:25:18 CEST

bliss@tigris.org wrote:

>Author: bliss
>Date: Sat Apr 3 09:22:48 2004
>New Revision: 9280
> trunk/subversion/include/svn_types.h
> trunk/subversion/libsvn_delta/text_delta.c
>Break apart the stream chunk size and the svndiff window size and
>reduce the chunk size to 8 kiB because it consumes less stack, is
>more TLB/cache friedly and is actually faster on some OSes. This
>also fixes "svn cat" for large files on Win32.
>* subversion/include/svn_types.h
> (SVN_STREAM_CHUNK_SIZE): Reduce from 100 kiB to 8 kiB.
> (SVN_MAX_OBJECT_SIZE): Remove strange FIXME.
I object to this last bit. It is not a "strange FIXME", it's a question
that needs to be answered before doing anything about it. Just removing
the comment doesn't answer the question, and you didn't even try to
analyse the pros and cons of making max-window-size the same as

>* subversion/libsvn_delta/text_delta.c
> (SVN_DELTA_WINDOW_SIZE): New define to use for the svndiff window
> size instead of SVN_STREAM_CHUNK_SIZE.
> (svn_txdelta, svn_txdelta_next_window): Use SVN_DELTA_WINDOW_SIZE.
I believe you missed at least fs-test. Your change modifies the
semantics of the file integrity tests.

Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Apr 3 18:23:50 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.