> Question: In the bucket split functions, a sanity check is performed on the 
> point
> passed in: if point is < 0 or > len, APR_EINVAL is returned.  What should be 
> the
> brigade-splitting equivalent of that?  Should an empty brigade be returned, or
> should the return value be NULL to make it more clear that there was an 
> error?  I'm
> leaning toward NULL to distinguish between error cases and the case where the 
> point
> == brigade len.
> 
> Thoughts?

Any error condition should return NULL in ap_brigade_split_offset.

Ryan

_______________________________________________________________________________
Ryan Bloom                              [EMAIL PROTECTED]
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------

Reply via email to