Differential D1962

Fix bunch of overrides warnings
ClosedPublic
Actions

Authored by bshah on Jun 20 2016, 3:17 PM.

Details

Reviewers
hein
Group Reviewers
Plasma
Commits
R871:b227b91df8df: Fix bunch of overrides warnings
R120:b227b91df8df: Fix bunch of overrides warnings
Summary

clang-check as well cppcheck complains about it

Test Plan

builds

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bshah updated this revision to Diff 4644.Jun 20 2016, 3:17 PM
bshah retitled this revision from to Fix bunch of overrides warnings.
bshah updated this object.
bshah edited the test plan for this revision. (Show Details)
bshah added reviewers: Plasma, hein.
Restricted Application added a project: Plasma. · View Herald TranscriptJun 20 2016, 3:17 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
hein accepted this revision.Jun 21 2016, 5:19 AM
hein edited edge metadata.
This revision is now accepted and ready to land.Jun 21 2016, 5:19 AM
Closed by commit R120:b227b91df8df: Fix bunch of overrides warnings (authored by bshah). · Explain WhyJun 21 2016, 5:22 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 4653

View Options

libtaskmanager/abstracttasksmodel.h

Loading...
View Options

libtaskmanager/flattentaskgroupsproxymodel.h

Loading...
View Options

libtaskmanager/taskfilterproxymodel.h

Loading...
View Options

libtaskmanager/taskgroupingproxymodel.h

Loading...
View Options

libtaskmanager/tasksmodel.h

Loading...