Differential D7917

[TasksModel] Use std::acumulate on the QHash directly
ClosedPublic
Actions

Authored by broulik on Sep 21 2017, 3:02 PM.

Details

Reviewers
hein
davidedmundson
Group Reviewers
Plasma
Commits
R120:602e93dfdb10: [TasksModel] Use std::acumulate on the QHash directly
Summary

begin() and end() operate on the hash values.
This avoids creating a temporary list just for this.

Test Plan

sumScore values still look similar

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Sep 21 2017, 3:02 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 21 2017, 3:02 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Sep 25 2017, 9:04 AM
This revision is now accepted and ready to land.Sep 25 2017, 9:04 AM
Closed by commit R120:602e93dfdb10: [TasksModel] Use std::acumulate on the QHash directly (authored by broulik). · Explain WhySep 25 2017, 12:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mlibtaskmanager/tasksmodel.cpp (3 lines)

Diff 19747

View Options

libtaskmanager/tasksmodel.cpp

Loading...