Comment 4 for bug 2027883

Revision history for this message
Durval Menezes (jm262) wrote :

> Doesn't it make more sense to just remove the else?

That would change the behavior of the code, in the sense that the following `if` would then be run regardless of the last one's result, instead of only if it's false.

Granted, in its context that would be mostly harmless (apart from a few wasted cycles), but in these cases (other people's code, etc) I subscribe to the "smallest change is the best change" school of thought.

Just my opinion, of course.