On 09/10/2012 11:49 AM, 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
It's pushed to master and 1.1. Yeah, this was a combination of new/old
usage -- I took an existing conditional and added to it. While I used &&
for my additions, I should have replaced the 'not' and 'or' at the same
time.
Scott