Re: svn commit: r372010 - /perl/modperl/trunk/src/modules/perl/modperl_cmd.c

2006-01-25 Thread Philippe M. Chiasson
Philip M. Gollucci wrote: -if(0 == strncasecmp(arg, +inherit, 8)) { +if (0 == strncasecmp(arg, +inherit, 8)) { modperl_cmd_options(parms, mconfig, +InheritSwitches); } else { Isn't the normal c idiom if (!strncasecmp() ? I don't think I've ever seen that form

Re: svn commit: r372010 - /perl/modperl/trunk/src/modules/perl/modperl_cmd.c

2006-01-25 Thread Philip M. Gollucci
A quick grep shows this is indeed the only instance, but there are quite a few similar usages in [httpd/modules/ldap/*, so I am not alone at the ASF ;-) Just for the record, I was disputing the correctness and am amazed how many responses I got from it. It just caught my eye as odd as I

Re: svn commit: r372010 - /perl/modperl/trunk/src/modules/perl/modperl_cmd.c

2006-01-25 Thread Philippe M. Chiasson
Philip M. Gollucci wrote: A quick grep shows this is indeed the only instance, but there are quite a few similar usages in [httpd/modules/ldap/*, so I am not alone at the ASF ;-) Just for the record, I was disputing the correctness and am amazed how many responses I got from it. It just

Re: svn commit: r372010 - /perl/modperl/trunk/src/modules/perl/modperl_cmd.c

2006-01-25 Thread Philip M. Gollucci
Just for the record, I was disputing the correctness and am amazed how many responses I got from it. It just caught my eye as odd as I personally have never used that form. EGAD that was supposed to be wasN'T disputing not was *sigh* Too much Percoset I guess. --

svn commit: r372010 - /perl/modperl/trunk/src/modules/perl/modperl_cmd.c

2006-01-24 Thread gozer
Author: gozer Date: Tue Jan 24 12:19:03 2006 New Revision: 372010 URL: http://svn.apache.org/viewcvs?rev=372010view=rev Log: Fix whitespace issue. WhiteSpace-Police: Stas Modified: perl/modperl/trunk/src/modules/perl/modperl_cmd.c Modified: