Differential D15968

kreadconfig: make the tool slightly more useful
Needs ReviewPublic
Actions

Authored by apol on Oct 5 2018, 4:41 PM.

Details

Reviewers
None
Group Reviewers
Frameworks
Summary

Include --help
Don't assume that globals are disabled because we're reading an existing file.
Allow the user to configure if cascading and globals are allowed

Test Plan

Ran the application locally

Diff Detail

Repository
R237 KConfig
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3525
Build 3543: arc lint + arc unit
apol created this revision.Oct 5 2018, 4:41 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 5 2018, 4:41 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
apol requested review of this revision.Oct 5 2018, 4:41 PM
apol updated this revision to Diff 42935.Oct 5 2018, 4:45 PM

Remove unneeded values

dfaure added a subscriber: dfaure.Oct 7 2018, 12:47 PM

Will you port the kreadconfig usage in startkde to add --ignore-globals --ignore-cascading in order to preserve performance?

Maybe it would be better to preserve behaviour compatibility and rather make globals and cascading "opt in"?

Revision Contents
Changeset List

PathPackages
Msrc/kreadconfig/kreadconfig.cpp (28 lines)

Diff 42935

View Options

src/kreadconfig/kreadconfig.cpp

Loading...