httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sander Striker" <stri...@apache.org>
Subject Code questions
Date Thu, 07 Mar 2002 19:21:37 GMT
Hi,

Some things I encountered while detabifying:


server/config.c:396
    return !!(cmd->limited & (AP_METHOD_BIT << methnum));
           ^^

Is that a typo or intentional?


server/core.c:661
    AP_DECLARE(const char *) ap_document_root(request_rec *r) /* Don't use this! */

If we shouldn't use it, why is it still here?


server/core.c:691
    /* Should probably just get rid of this... the only code that cares is
     * part of the core anyway (and in fact, it isn't publicised to other
     * modules).
     */

Read the comment.


server/listen.c:320
    /*free(lr);*/

Can this go?  Was there a future purpose to this call,
or was it just old code commented out?

Thanks,

Sander



Mime
View raw message