Remvoe AP_END_OF_BRIGADE. This definition didn't really mean anything,

because it didn't signify the end of the brigade, and it didn't signify
the end of a response.  It ended up meaning flush the data, which can be
detemined with bucket type checks.  This also makes the FLUSH and EOS
buckets have a 0 length, which makes a lot of code cleaner, because now we
can just add lengths and be sure they are all >=0.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@86851 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/modules/experimental/mod_charset_lite.c b/modules/experimental/mod_charset_lite.c
index a6f0172de7f72fe9e2b69ff27f57b2dc68ff08db..72da9e4dbf9e917e7f6c57d687f7a6d96dded473 100644 (file)
--- a/modules/experimental/mod_charset_lite.c
+++ b/modules/experimental/mod_charset_lite.c
@@ -881,7 +881,7 @@ static apr_status_t xlate_out_filter(ap_filter_t *f, ap_bucket_brigade *bb)
             }
             if (AP_BUCKET_IS_EOS(dptr)) {
                 done = 1;
-                cur_len = AP_END_OF_BRIGADE; /* XXX yuck, but that tells us to send
+                cur_len = -1; /* XXX yuck, but that tells us to send
                                  * eos down; when we minimize our bb construction
                                  * we'll fix this crap */
                 if (ctx->saved) {
@@ -962,7 +962,7 @@ static apr_status_t xlate_out_filter(ap_filter_t *f, ap_bucket_brigade *bb)
         }
     }
     if (rv == APR_SUCCESS) {
-        if (cur_len == AP_END_OF_BRIGADE) {
+        if (cur_len == -1) {
             rv = send_eos(f);
         }
     }