That code is incorrect right now.  I am close to re-writing it with 
apr_brigade_partition,
but that caused some bugs yesterday.  The real problem is that the code in that 
filter
is trying to do too much, and it isn't taking advantage of the mechanism we 
provide.

All of the types are correct though.  I can go over the logic behind it if you 
would like.

Ryan

On Tuesday 07 August 2001 22:05, Ian Holsman wrote:
> should the 'point' parameter in the bucket split (shared/notimpl/ & simple)
> be a 'apr_off_t' or a 'apr_size_t' ?
>
> This seems to be the only warning generated from the Win2k build
>
> http_protocol.c
> c:\src\apache\httpd-2.0\modules\http\http_protocol.c(678) : warning
> C4244: 'function' : conversion from '__int64 ' to 'unsigned int ',
> possible loss of data
>
> the offending parameter is 'readbytes'
>
> apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
> ap_input_mode_t mode, apr_off_t *readbytes)
>
>
> ..Ian

-- 

_____________________________________________________________________________
Ryan Bloom                              [EMAIL PROTECTED]
Covalent Technologies                   [EMAIL PROTECTED]
-----------------------------------------------------------------------------

Reply via email to