https://bugzilla.novell.com/show_bug.cgi?id=409150
User ma@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=409150#c2
Michael Andres changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ma@novell.com
--- Comment #2 from Michael Andres 2008-07-16 04:46:08 MDT ---
But does it need to be that complicated? Isn't it:
// needed
if ( selectable->fate() == Selectable::TO_INSTALL
|| ( selectable->hasCandidateObj()
&& selectable->candidateObj().isBroken() ) )
// unneeded
if ( selectable->fate() != Selectable::TO_INSTALL
&& ( selectable->hasCandidateObj()
&& ! selectable->candidateObj().isBroken() ) )
Remember that there are 3 mutual exclusive states:
! isRelevant()
isSatisfied()
isBroken()
Thus
isRelevant() && ! isSatisfied() means isBroken()
! isRelevant() || isSatisfied() means ! isBroken()
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.