Differential D9689 Diff 24838 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​ unsigned long timestamp = QDateTime::currentMSecsSinceEpoch();
807​ // Only add an entry if the latest one is older than MIN_HIST_AGE
808​ if(!history.size() || timestamp - history.constLast().timestamp > MIN_HIST_AGE) {
broulikUnsubmitted
Not Done

isEmpty()

broulik: `isEmpty()`
809​ if(history.size() == MAX_HIST_ENTRIES)
broulikUnsubmitted
Not Done

Braces also for one-line statements

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