Ruediger Pluem wrote:

Hm, why don't we do the ap_meets_conditions test here like in the quick handler

    rv = ap_meets_conditions(r);
    if (rv != OK) {

        /* Return cached status. */
        return rv;
    }

This code path is a normal handler, we don't need to. ap_meets_conditions() is already done by the core.

Plus we have one new warning:

mod_cache.c: In function 'cache_handler':
mod_cache.c:333: warning: unused variable 'auth'

Fixed in 821301.

Regards,
Graham
--

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to