httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r1161661 - /httpd/httpd/trunk/modules/http/byterange_filter.c
Date Fri, 26 Aug 2011 12:38:20 GMT

On Aug 26, 2011, at 8:31 AM, Jim Jagielski wrote:
> 
> Agreed… The issue is that to merge, you need to parse… We could have
> ap_set_byterange() also return a set of start/stop points (in an
> array), so that the filter can work with that and not bother with rereading
> r->range. In fact, I think that makes the most sense… That is, adjust
> ap_set_byterange() to create the modified, parsed r->range, a return status
> and the start/stop array. The filter then goes thru that array.

So ap_set_byterange(request_rec *r, apr_off_t clen, apr_array_header_t *indexes)
will exist and parse_byterange() will go away…

Sound OK?
Mime
View raw message