Differential D16859

Rename config loading function
ClosedPublic
Actions

Authored by romangg on Nov 13 2018, 11:12 AM.

Tags

Subscribers

Details

Reviewers

Group Reviewers

Commits

R104:af6da1146ef6: Rename config loading function

Summary

Make it more explicit what it does.

Test Plan

Serializer test passes.

Diff Detail

Repository

Branch

configReadCleanup

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 4897
Build 4915: arc lint + arc unit

romangg created this revision.Nov 13 2018, 11:12 AM

Restricted Application added a project: Plasma. · View Herald TranscriptNov 13 2018, 11:12 AM

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

romangg requested review of this revision.Nov 13 2018, 11:12 AM

Harbormaster completed remote builds in B4897: Diff 45406.Nov 13 2018, 11:12 AM

Remove unrelated change

Harbormaster completed remote builds in B4898: Diff 45407.Nov 13 2018, 11:14 AM

davidedmundson accepted this revision.Nov 15 2018, 10:30 AM

This revision is now accepted and ready to land.Nov 15 2018, 10:30 AM

Closed by commit R104:af6da1146ef6: Rename config loading function (authored by romangg). · Explain WhyNov 15 2018, 4:22 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			kded/daemon.cpp (4 lines)
M			kded/serializer.h (2 lines)
M			kded/serializer.cpp (5 lines)
M			tests/kded/serializertest.cpp (28 lines)

Diff 45406

kded/daemon.cpp

Loading...

kded/serializer.h

Loading...

kded/serializer.cpp

Loading...

tests/kded/serializertest.cpp

Loading...