httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: apache-2.0/src/main http_core.c
Date Wed, 08 Nov 2000 03:05:39 GMT
hehe... beat me to it. Went out for some errands and realized that I had
checked r->clength but not ap_set_content_length(). D'oh!

Thanks for the fix, Ryan.

Cheers,
-g

On Wed, Nov 08, 2000 at 02:46:05AM -0000, rbb@locus.apache.org wrote:
> rbb         00/11/07 18:46:05
> 
>   Modified:    src/main http_core.c
>   Log:
>   Fix a compile break.  ap_set_content_length is a void function now, so we
>   can't check the return code.
>   
>   Revision  Changes    Path
>   1.205     +2 -2      apache-2.0/src/main/http_core.c
>   
>   Index: http_core.c
>   ===================================================================
>   RCS file: /home/cvs/apache-2.0/src/main/http_core.c,v
>   retrieving revision 1.204
>   retrieving revision 1.205
>   diff -u -r1.204 -r1.205
>   --- http_core.c	2000/11/07 20:21:52	1.204
>   +++ http_core.c	2000/11/08 02:46:00	1.205
>   @@ -2966,8 +2966,8 @@
>        ap_set_last_modified(r);
>        ap_set_etag(r);
>        apr_table_setn(r->headers_out, "Accept-Ranges", "bytes");
>   -    if (((errstatus = ap_meets_conditions(r)) != OK)
>   -	|| (errstatus = ap_set_content_length(r, r->finfo.size))) {
>   +    ap_set_content_length(r, r->finfo.size);
>   +    if ((errstatus = ap_meets_conditions(r)) != OK) {
>            apr_close(fd);
>            return errstatus;
>        }
>   
>   
>   

-- 
Greg Stein, http://www.lyra.org/

Mime
View raw message