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
