> > Filters shouldn't return apr_status_t's, because there is nothing that > > the core can do with a status code. > > Okay then... core_output_filter should not be returning APR_SUCCESS, and > its return type should not be apr_status_t. I had two options, and picked > the other. Both are fine. :)
I agree, filters should be returning int, and they should return HTTP_OK or HTTP_RANDOM_STATUS_CODE. Now, we just have to make the change. :-) Ryan
