diff options
author | Rakesh Pandit <rakesh@fedoraproject.org> | 2010-04-14 16:44:52 +0530 |
---|---|---|
committer | Panu Matilainen <pmatilai@redhat.com> | 2010-04-21 12:03:09 +0300 |
commit | 8cc740c43f28b43cd34894d7c0d47b48ce5f4f91 (patch) | |
tree | a7eaeb31f97976c81f761838987d5e9d67bb51cb /lib/rpmps.c | |
parent | c0ab315e71c46e36de038dde4a32ebe6eba04c41 (diff) | |
download | rpm-8cc740c43f28b43cd34894d7c0d47b48ce5f4f91.tar.gz |
Eliminate unused "known problems" based filtering from rpmtsRun()
- This mechanism has never been used by anything at all, which suggests
its not a particularly useful feature
- Removing unused rpmpsTrim() from rpmps
- Marking okProbs parameter of rpmtsRun() function as unused to avoid
unnecessary breakage
Diffstat (limited to 'lib/rpmps.c')
-rw-r--r-- | lib/rpmps.c | 37 |
1 files changed, 0 insertions, 37 deletions
diff --git a/lib/rpmps.c b/lib/rpmps.c index 7f089af77..7107e3922 100644 --- a/lib/rpmps.c +++ b/lib/rpmps.c @@ -139,43 +139,6 @@ void rpmpsAppendProblem(rpmps ps, rpmProblem prob) ps->numProblems++; } -/* XXX TODO: implement with iterators */ -int rpmpsTrim(rpmps ps, rpmps filter) -{ - rpmProblem *t; - rpmProblem *f; - int gotProblems = 0; - - if (ps == NULL || ps->numProblems == 0) - return 0; - - if (filter == NULL) - return (ps->numProblems == 0 ? 0 : 1); - - t = ps->probs; - f = filter->probs; - - while ((f - filter->probs) < filter->numProblems) { - while ((t - ps->probs) < ps->numProblems) { - if (rpmProblemCompare(*f, *t) == 0) - break; - t++; - gotProblems = 1; - } - - /* XXX This can't happen, but let's be sane in case it does. */ - if ((t - ps->probs) == ps->numProblems) - break; - - t++, f++; - } - - if ((t - ps->probs) < ps->numProblems) - gotProblems = 1; - - return gotProblems; -} - /* * TODO: filter out duplicates while merging. Also horribly inefficient... */ int rpmpsMerge(rpmps dest, rpmps src) |