diff options
author | Panu Matilainen <pmatilai@redhat.com> | 2009-08-31 12:15:16 +0300 |
---|---|---|
committer | Panu Matilainen <pmatilai@redhat.com> | 2009-08-31 12:15:16 +0300 |
commit | 6585c6485ab318c836d16edc069bc1be01e0e425 (patch) | |
tree | 826c96247007a74cbe776348aac004df94ab5ce8 /lib/rpmps.c | |
parent | feece1341e32efeafa1fbdeb893e33629147603e (diff) | |
download | rpm-6585c6485ab318c836d16edc069bc1be01e0e425.tar.gz rpm-6585c6485ab318c836d16edc069bc1be01e0e425.tar.bz2 rpm-6585c6485ab318c836d16edc069bc1be01e0e425.zip |
sameProblem() return value is illogical, reverse it
Diffstat (limited to 'lib/rpmps.c')
-rw-r--r-- | lib/rpmps.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/lib/rpmps.c b/lib/rpmps.c index 54ab1921a..93a86cfd9 100644 --- a/lib/rpmps.c +++ b/lib/rpmps.c @@ -379,21 +379,21 @@ char * rpmProblemString(const rpmProblem prob) static int sameProblem(const rpmProblem ap, const rpmProblem bp) { if (ap->type != bp->type) - return 1; + return 0; if (ap->pkgNEVR) if (bp->pkgNEVR && strcmp(ap->pkgNEVR, bp->pkgNEVR)) - return 1; + return 0; if (ap->altNEVR) if (bp->altNEVR && strcmp(ap->altNEVR, bp->altNEVR)) - return 1; + return 0; if (ap->str1) if (bp->str1 && strcmp(ap->str1, bp->str1)) - return 1; + return 0; if (ap->num1 != bp->num1) - return 1; + return 0; - return 0; + return 1; } void rpmpsPrint(FILE *fp, rpmps ps) @@ -419,7 +419,7 @@ void rpmpsPrint(FILE *fp, rpmps ps) rpmpsi psif = rpmpsInitIterator(ps); /* Filter already displayed problems. */ while ((j = rpmpsNextIterator(psif)) < i) { - if (!sameProblem(p, rpmpsGetProblem(psif))) + if (sameProblem(p, rpmpsGetProblem(psif))) break; } rpmpsFreeIterator(psif); |