From 3632294483dd1eb8f26ed390290cf39580bdffc7 Mon Sep 17 00:00:00 2001 From: Panu Matilainen Date: Fri, 4 Mar 2011 13:46:06 +0200 Subject: [PATCH] Remove pointless return code from handleInstInstalledFiles() - There are no errors to return here... so it might as well be void --- lib/transaction.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/transaction.c b/lib/transaction.c index 628e4ea..d16a7bd 100644 --- a/lib/transaction.c +++ b/lib/transaction.c @@ -324,7 +324,7 @@ static uint64_t countFiles(rpmts ts) * @param reportConflicts */ /* XXX only ts->{probs,rpmdb} modified */ -static int handleInstInstalledFile(const rpmts ts, rpmte p, rpmfi fi, +static void handleInstInstalledFile(const rpmts ts, rpmte p, rpmfi fi, Header otherHeader, rpmfi otherFi, int beingRemoved) { @@ -333,7 +333,7 @@ static int handleInstInstalledFile(const rpmts ts, rpmte p, rpmfi fi, int isCfgFile = ((rpmfiFFlags(otherFi) | rpmfiFFlags(fi)) & RPMFILE_CONFIG); if (XFA_SKIPPING(rpmfsGetAction(fs, fx))) - return 0; + return; if (rpmfiCompare(otherFi, fi)) { rpm_color_t tscolor = rpmtsColor(ts); @@ -377,8 +377,6 @@ static int handleInstInstalledFile(const rpmts ts, rpmte p, rpmfi fi, rpmfsSetAction(fs, fx, action); } rpmfiSetFReplacedSize(fi, rpmfiFSize(otherFi)); - - return 0; } /** @@ -912,7 +910,6 @@ void checkInstalledFiles(rpmts ts, uint64_t fileCount, rpmFpHash ht, fingerPrint rpmfs fs; rpmfi otherFi=NULL; int j; - int xx; unsigned int fileNum; const char * oldDir; @@ -994,7 +991,7 @@ void checkInstalledFiles(rpmts ts, uint64_t fileCount, rpmFpHash ht, fingerPrint } rpmfiSetFX(fi, recs[j].fileno); rpmfiSetFX(otherFi, fileNum); - xx = handleInstInstalledFile(ts, p, fi, h, otherFi, beingRemoved); + handleInstInstalledFile(ts, p, fi, h, otherFi, beingRemoved); break; case TR_REMOVED: if (!beingRemoved) { -- 2.7.4