Re: [PATCH] Make mod_authz_svn to apply the authz checks against upper/lowercased usernames

On Thu, 13 Mar 2008, Kamesh Jayachandran wrote:

>
> >I have some simplifications for that (attached, untested).
> >
> > while (*c) {
> >- *c = convert_func(*c);
> >+ *c = (to_uppercase ? apr_toupper(*c) : apr_tolower(*c));
> > ++c;
>
>
> I thought of this approach, but stayed away from it as 'to_upper check'
> happens for every character in a loop.

So? It's a single boolean check. The gain in simplicity far outweighs any
micro-optimization.