Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> [...]
>
>>Yes! That's also another _very_ good reason to type comparaisons with the
>>constant
>>on the left side. I always try to write it in that order, and by now, my brain
>>is fully adjusted to it.
>>
>>if("Inherit" eq $var) {
>>elsif ("
Philippe M. Chiasson wrote:
[...]
Yes! That's also another _very_ good reason to type comparaisons with the
constant
on the left side. I always try to write it in that order, and by now, my brain
is fully adjusted to it.
if("Inherit" eq $var) {
elsif ("Enable" eq $var) {
elsif ("Disable" eq
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.
--
--
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 jus
Malcolm J Harwood wrote:
> On Wednesday 25 January 2006 11:29 pm, Philip M. Gollucci wrote:
>
>
>>>-if(0 == strncasecmp(arg, "+inherit", 8)) {
>>>+if (0 == strncasecmp(arg, "+inherit", 8)) {
>>>modperl_cmd_options(parms, mconfig, "+InheritSwitches");
>>>}
>>>else {
>>
>>Is
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 personal
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 ev
On Wednesday 25 January 2006 11:29 pm, 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 (!
-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 (though correct)