On 09/10/2012 05:49 PM, Jason Guiditta wrote:
On 10/09/12 11:40 -0400, Scott Seago wrote:
>
https://bugzilla.redhat.com/show_bug.cgi?id=798516
>
> Apparently on f16/ruby 1.8, the conditional block with 'or' and 'not'
is
> not working as expected. I don't know if it's a ruby bug or something
> else
> but it doesn't matter in this case -- using '!' and '||' is the
right
> way to do the conditional anyway.
>
Visual, ACK, feel free to push this. I don't like seeing our rampant
usage of and|or|not anyway, I think it will leads to lots of bugs like
this due to the unexpected changes in order of precendence. I suggest
we try to move to the more standard operators throughout the codebase
(I'd go so far as to NACK an introductions of new AND|OR|NOT usage).
-j
Yeah, I totally agree. These operators are more like control-flow
modifiers and should not used as logical operators.
How about extending our coding standards with a new paragraph on
restricting the usage of AND|OR|NOT?
Imre