You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Cliff Woolley <jw...@virginia.edu> on 2002/07/02 20:32:41 UTC

Re: cvs commit: apr-util/buckets apr_brigade.c

On 2 Jul 2002 wrowe@apache.org wrote:

> wrowe       2002/07/02 11:20:57
>
>   Modified:    buckets  apr_brigade.c
>   Log:
>     New emit on win32.  Since this is a heap bucket, we are always dealing
>     in start offsets that fit in apr_size_t.  This doesn't need the file
>     sized apr_off_t resolution.
>
>   -        remaining = h->alloc_len - (e->length + e->start);
>   +        /* HEAP bucket start offsets are always in-memory, safe to cast */
>   +        remaining = h->alloc_len - (e->length + (apr_size_t)e->start);
>            buf = h->base + e->start + e->length;
>        }

Okay, so why on earth would the remaining= line need that cast when the
buf= line doesn't?

Just curious.

--Cliff