You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2014/04/20 20:02:41 UTC
svn commit: r1588812 -
/subversion/trunk/subversion/libsvn_fs_fs/transaction.c
Author: stefan2
Date: Sun Apr 20 18:02:40 2014
New Revision: 1588812
URL: http://svn.apache.org/r1588812
Log:
* subversion/libsvn_fs_fs/transaction.c
(fnv1a_checksum_on_file_range): Use our standard stream / file data block
size instead of some local constant.
Found by: ivan
Modified:
subversion/trunk/subversion/libsvn_fs_fs/transaction.c
Modified: subversion/trunk/subversion/libsvn_fs_fs/transaction.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/transaction.c?rev=1588812&r1=1588811&r2=1588812&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/transaction.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/transaction.c Sun Apr 20 18:02:40 2014
@@ -3567,7 +3567,7 @@ fnv1a_checksum_on_file_range(apr_uint32_
apr_off_t size,
apr_pool_t *pool)
{
- char buffer[4096];
+ char buffer[SVN__STREAM_CHUNK_SIZE];
svn_checksum_ctx_t *checksum_ctx
= svn_checksum_ctx_create(svn_checksum_fnv1a_32x4, pool);
Re: svn commit: r1588812 - /subversion/trunk/subversion/libsvn_fs_fs/transaction.c
Posted by Ivan Zhakov <iv...@visualsvn.com>.
On 20 April 2014 22:02, <st...@apache.org> wrote:
> Author: stefan2
> Date: Sun Apr 20 18:02:40 2014
> New Revision: 1588812
>
> URL: http://svn.apache.org/r1588812
> Log:
> * subversion/libsvn_fs_fs/transaction.c
> (fnv1a_checksum_on_file_range): Use our standard stream / file data block
> size instead of some local constant.
>
> Found by: ivan
>
> Modified:
> subversion/trunk/subversion/libsvn_fs_fs/transaction.c
>
> Modified: subversion/trunk/subversion/libsvn_fs_fs/transaction.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/transaction.c?rev=1588812&r1=1588811&r2=1588812&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/libsvn_fs_fs/transaction.c (original)
> +++ subversion/trunk/subversion/libsvn_fs_fs/transaction.c Sun Apr 20 18:02:40 2014
> @@ -3567,7 +3567,7 @@ fnv1a_checksum_on_file_range(apr_uint32_
> apr_off_t size,
> apr_pool_t *pool)
> {
> - char buffer[4096];
> + char buffer[SVN__STREAM_CHUNK_SIZE];
>
Allocating 64k on stack is not good thing imho. Currently we always
allocate such buffers from pool and this case looks like another
premature optimization.
---
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com