On Fri, Sep 12, 2014 at 4:57 PM, Yann Ylavic <ylavic....@gmail.com> wrote: > AP_DECLARE(int) ap_filter_reinstate_brigade(ap_filter_t *f, > ap_filter_aside_t *aside, > apr_bucket_brigade *bb, > apr_bucket **flush_upto) > { > apr_bucket *bucket, *next; > apr_size_t bytes_in_brigade, non_file_bytes_in_brigade; > int eor_buckets_in_brigade, morphing_bucket_in_brigade; > int loglevel = ap_get_conn_module_loglevel(f->c, APLOG_MODULE_INDEX); > int empty = APR_BRIGADE_EMPTY(bb); > > if ((*aside)->bb && !APR_BRIGADE_EMPTY((*aside)->bb)) { > APR_BRIGADE_PREPEND(bb, (*aside)->bb); > f->c->data_in_output_filters--; > if (empty) { > return 1; > } > }
The (empty) check could even be done outside the block, in any case. > > ... > }