You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Garrett Rooney <ro...@electricjellyfish.net> on 2006/01/14 01:19:28 UTC

Re: svn commit: r368929 - in /httpd/httpd/trunk/modules/aaa: config.m4 mod_auth.h mod_authz_core.c mod_authz_default.c

On 1/13/06, bnicholes@apache.org <bn...@apache.org> wrote:

> --- httpd/httpd/trunk/modules/aaa/config.m4 (original)
> +++ httpd/httpd/trunk/modules/aaa/config.m4 Fri Jan 13 16:13:22 2006
> @@ -48,6 +48,10 @@
>  dnl keep the bad guys out.
>  APACHE_MODULE(authz_default, authorization control backstopper, , , yes)
>
> +dnl - and just in case all of the above punt; a default handler to
> +dnl keep the bad guys out.
> +APACHE_MODULE(access_compat, mod_access compatibility, , , most)
> +

This comment seems wrong...

> --- httpd/httpd/trunk/modules/aaa/mod_authz_core.c (original)
> +++ httpd/httpd/trunk/modules/aaa/mod_authz_core.c Fri Jan 13 16:13:22 2006
> @@ -101,6 +101,8 @@
>      authz_provider_list *providers;
>      authz_request_state req_state;
>      int req_state_level;
> +//    int some_authz;
> +//    char *path;

And the c++ style comments in this file won't work everywhere... 
Actually, I'm curious why this stuff is being commented out anyway,
instead of just deleted.  We do have version control, after all, we
can get them back if we want to ;-)

-garrett