On Nov 15, 2007, at 7:02 AM, Plüm, Rüdiger, VF-Group wrote:


-    if (APR_BRIGADE_SENTINEL(ctx->bb)) {
+    if (APR_BRIGADE_EMPTY(ctx->bb)) {
+        *len = 0;
+        return APR_EOF;
+    }
+
+    if (APR_BUCKET_IS_EOS(APR_BRIGADE_FIRST(ctx->bb)) ||
+        AP_BUCKET_IS_EOR(APR_BRIGADE_FIRST(ctx->bb))) {
         *len = 0;
         return APR_EOF;

Why do we not put this in one if block like?


LOL... I saw the commit message and thought the same thing and
actually changed it before I actually checked dev@ :) :)


Reply via email to