You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Julian Foad <ju...@wandisco.com> on 2011/05/09 18:40:58 UTC

Re: svn commit: r1101091 - /subversion/trunk/subversion/libsvn_subr/cache-membuffer.c

stefan2@apache.org wrote:
> * subversion/libsvn_subr/cache-membuffer.c
>   (MIN_SEGMENT_SIZE): ensure that cache sizes > 4GB work
>   (svn_cache__membuffer_cache_create): formatting fixes; add missing cast

>    /* allocate cache as an array of segments / cache objects */
> -  c = apr_palloc(pool, segment_count * sizeof(*c));
> +  c = (svn_membuffer_t *)apr_palloc(pool, segment_count * sizeof(*c));

Hi Stefan.  In contrast to C++, in C it's a handy convenience that we
don't need to cast "void *" pointers before assigning them.  This is the
only such cast in the code base.

- Julian