You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2010/08/22 18:07:19 UTC

Re: svn commit: r987869 - /subversion/branches/performance/subversion/libsvn_fs_fs/temp_serial izer.c

Stefan Sperling wrote on Sun, Aug 22, 2010 at 13:44:07 +0200:
> On Sun, Aug 22, 2010 at 11:37:38AM -0000, stefan2@apache.org wrote:
> > +  apr_size_t entries_len = count * sizeof(svn_fs_dirent_t*[1]);
> 

So how would this be parsed?  Is it parsed as

	sizeof( (type *) [1] )

?

> Do you really want the second asterisk in that line?
> It looks like you really want this:
> 
>  +  apr_size_t entries_len = count * sizeof(svn_fs_dirent_t[1]);