Differential D28365

[krunner] Avoid (another) pointless config load on start
ClosedPublic
Actions

Authored by davidedmundson on Mar 27 2020, 10:21 PM.

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R120:1c48e351c646: [krunner] Avoid reparsing the config load on start

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.
Restricted Application added a project: Plasma. · View Herald TranscriptMar 27 2020, 10:21 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Mar 27 2020, 10:21 PM
apol accepted this revision.Mar 28 2020, 12:24 AM
apol added a subscriber: apol.
apol added inline comments.
krunner/view.cpp
110 ↗(On Diff #78706)

Using 2 lambdas looks unnecessary, make a reparseConfig private slot?

This revision is now accepted and ready to land.Mar 28 2020, 12:24 AM
Closed by commit R120:1c48e351c646: [krunner] Avoid reparsing the config load on start (authored by davidedmundson). · Explain WhyApr 3 2020, 11:33 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkrunner/view.h (2 lines)
Mkrunner/view.cpp (15 lines)

Diff 79205

View Options

krunner/view.h

Loading...
View Options

krunner/view.cpp

Loading...