Changeset View
Changeset View
Standalone View
Standalone View
libdiscover/backends/PackageKitBackend/PackageKitBackend.cpp
| Show First 20 Lines • Show All 577 Lines • ▼ Show 20 Line(s) | |||||
| 578 | } | 578 | } | ||
| 579 | 579 | | |||
| 580 | void PackageKitBackend::addPackageToUpdate(PackageKit::Transaction::Info info, const QString& packageId, const QString& summary) | 580 | void PackageKitBackend::addPackageToUpdate(PackageKit::Transaction::Info info, const QString& packageId, const QString& summary) | ||
| 581 | { | 581 | { | ||
| 582 | if (info == PackageKit::Transaction::InfoBlocked) { | 582 | if (info == PackageKit::Transaction::InfoBlocked) { | ||
| 583 | return; | 583 | return; | ||
| 584 | } | 584 | } | ||
| 585 | 585 | | |||
| 586 | if (info == PackageKit::Transaction::InfoRemoving || info == PackageKit::Transaction::InfoObsoleting) { | ||||
apol: Could you drop the unnecessary parenthesis? | |||||
| 587 | // Don't try updating packages which need to be removed | ||||
| 588 | return; | ||||
| 589 | } | ||||
| 590 | | ||||
| 586 | if (info == PackageKit::Transaction::InfoSecurity) | 591 | if (info == PackageKit::Transaction::InfoSecurity) | ||
| 587 | m_hasSecurityUpdates = true; | 592 | m_hasSecurityUpdates = true; | ||
| 588 | 593 | | |||
| 589 | m_updatesPackageId += packageId; | 594 | m_updatesPackageId += packageId; | ||
| 590 | addPackage(info, packageId, summary, true); | 595 | addPackage(info, packageId, summary, true); | ||
| 591 | } | 596 | } | ||
| 592 | 597 | | |||
| 593 | void PackageKitBackend::getUpdatesFinished(PackageKit::Transaction::Exit, uint) | 598 | void PackageKitBackend::getUpdatesFinished(PackageKit::Transaction::Exit, uint) | ||
| ▲ Show 20 Lines • Show All 90 Lines • Show Last 20 Lines | |||||
Could you drop the unnecessary parenthesis?