Differential D9689 Diff 24803 processui/ProcessModel.cpp

Changeset View

Standalone View

View Options

processui/ProcessModel.cpp

Show First 20 Lines • Show All 791 Lines • ▼ Show 20 Line(s)734 } else {
792 } 792​ }
793 if(process->changes() & KSysGuard::Process::IO) { 793​ if(process->changes() & KSysGuard::Process::IO) {
794 totalUpdated++; 794​ totalUpdated++;
795 QModelIndex index = q->createIndex(row, ProcessModel::HeadingIoRead, process); 795​ QModelIndex index = q->createIndex(row, ProcessModel::HeadingIoRead, process);
796 emit q->dataChanged(index, index); 796​ emit q->dataChanged(index, index);
797 index = q->createIndex(row, ProcessModel::HeadingIoWrite, process); 797​ index = q->createIndex(row, ProcessModel::HeadingIoWrite, process);
798 emit q->dataChanged(index, index); 798​ emit q->dataChanged(index, index);
799 } 799​ }
800​
801​ /* Normally this would only be called if changes() tells
802​ * us to. We need to update the timestamp even if the value
803​ * didn't change though. */
804​ if(mMapProcessCPUHistory.contains(process)) {
broulikUnsubmitted
Not Done

Use find() to avoid double lookup (contains+operator[])

broulik: Use `find()` to avoid double lookup (contains+operator[])
805​ auto &history = mMapProcessCPUHistory[process];
806​ float usage = (process->totalUserUsage() + process->totalSysUsage()) / (100.0f * mNumProcessorCores);
807​ if(history.size()) {
808​ auto &last = history.last();
broulikUnsubmitted
Not Done

isEmpty()

broulik: `isEmpty()`
809​ if(last.value == usage)
broulikUnsubmitted
Not Done

Braces also for one-line statements

broulik: Braces also for one-line statements
810​ last.timestamp = static_cast<unsigned long>(QDateTime::currentMSecsSinceEpoch());
811​ else
812​ history.push_back({static_cast<unsigned long>(QDateTime::currentMSecsSinceEpoch()), usage});
813​ }
814​ else
815​ history.push_back({static_cast<unsigned long>(QDateTime::currentMSecsSinceEpoch()), usage});
816​
817​ if(history.size() > MAX_HIST_ENTRIES)
818​ history.removeFirst();
819​ }
800 } 820​ }
801} 821​}
802 822​
803void ProcessModelPrivate::beginInsertRow( KSysGuard::Process *process) 823​void ProcessModelPrivate::beginInsertRow( KSysGuard::Process *process)
804{ 824​{
805 Q_ASSERT(process); 825​ Q_ASSERT(process);
806 Q_ASSERT(!mRemovingRow); 826​ Q_ASSERT(!mRemovingRow);
807 Q_ASSERT(!mInsertingRow); 827​ Q_ASSERT(!mInsertingRow);
Show All 29 Lines
837{ 857​{
838 Q_ASSERT(process); 858​ Q_ASSERT(process);
839 Q_ASSERT(process->pid() >= 0); 859​ Q_ASSERT(process->pid() >= 0);
840 Q_ASSERT(!mRemovingRow); 860​ Q_ASSERT(!mRemovingRow);
841 Q_ASSERT(!mInsertingRow); 861​ Q_ASSERT(!mInsertingRow);
842 Q_ASSERT(!mMovingRow); 862​ Q_ASSERT(!mMovingRow);
843 mRemovingRow = true; 863​ mRemovingRow = true;
844 864​
865​ mMapProcessCPUHistory.remove(process);
866​
845 if(mSimple) { 867​ if(mSimple) {
846 return q->beginRemoveRows(QModelIndex(), process->index(), process->index()); 868​ return q->beginRemoveRows(QModelIndex(), process->index(), process->index());
847 } else { 869​ } else {
848 int row = process->parent()->children().indexOf(process); 870​ int row = process->parent()->children().indexOf(process);
849 if(row == -1) { 871​ if(row == -1) {
850 qCDebug(LIBKSYSGUARD) << "A serious problem occurred in remove row."; 872​ qCDebug(LIBKSYSGUARD) << "A serious problem occurred in remove row.";
851 mRemovingRow = false; 873​ mRemovingRow = false;
852 return; 874​ return;
▲ Show 20 Lines • Show All 904 Lines • ▼ Show 20 Line(s)1760 switch(index.column()) {
1757 case HeadingSharedMemory: 1779​ case HeadingSharedMemory:
1758 if(process->vmURSS() == -1 || d->mMemTotal <= 0) 1780​ if(process->vmURSS() == -1 || d->mMemTotal <= 0)
1759 return -1; 1781​ return -1;
1760 return float(process->vmRSS() - process->vmURSS())/d->mMemTotal; 1782​ return float(process->vmRSS() - process->vmURSS())/d->mMemTotal;
1761 default: 1783​ default:
1762 return -1; 1784​ return -1;
1763 } 1785​ }
1764 } 1786​ }
1787​ case PercentageHistoryRole: {
1788​ KSysGuard::Process *process = reinterpret_cast< KSysGuard::Process * > (index.internalPointer());
1789​ Q_CHECK_PTR(process);
1790​ switch(index.column()) {
1791​ case HeadingCPUUsage: {
1792​ auto it = d->mMapProcessCPUHistory.find(process);
1793​ if (it == d->mMapProcessCPUHistory.end()) {
1794​ it = d->mMapProcessCPUHistory.insert(process, {});
1795​ it->reserve(ProcessModelPrivate::MAX_HIST_ENTRIES);
1796​ }
1797​ return QVariant::fromValue(*it);
1798​ }
1799​ default: {}
1800​ }
1801​ return QVariant::fromValue(QVector<PercentageHistoryEntry>{});
1802​ }
1765 case Qt::DecorationRole: { 1803​ case Qt::DecorationRole: {
1766 if(index.column() == HeadingName) { 1804​ if(index.column() == HeadingName) {
1767#if HAVE_X11 1805​#if HAVE_X11
1768 KSysGuard::Process *process = reinterpret_cast< KSysGuard::Process * > (index.internalPointer()); 1806​ KSysGuard::Process *process = reinterpret_cast< KSysGuard::Process * > (index.internalPointer());
1769 if(!process->hasManagedGuiWindow()) { 1807​ if(!process->hasManagedGuiWindow()) {
1770 if(d->mSimple) //When not in tree mode, we need to pad the name column where we do not have an icon 1808​ if(d->mSimple) //When not in tree mode, we need to pad the name column where we do not have an icon
1771 return QIcon(d->mBlankPixmap); 1809​ return QIcon(d->mBlankPixmap);
1772 else //When in tree mode, the padding looks bad, so do not pad in this case 1810​ else //When in tree mode, the padding looks bad, so do not pad in this case
▲ Show 20 Lines • Show All 389 Lines • Show Last 20 Lines