D25791: Fix writeFlags with KConfigCompilerSignallingItem
David Edmundson
noreply at phabricator.kde.org
Fri Dec 6 19:00:49 GMT 2019
davidedmundson created this revision.
davidedmundson added a reviewer: ervin.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.
REVISION SUMMARY
KConfigCompilerSignallingItem both inherits KConfigSkeletonItem and
internally is powered by a separate KConfigSkeletonItem
The generated code calls setWriteFlags on the outer
KConfigCompilerSignallingItem instance, but the real writing is
performed by the internal version. We need to set the flags in the right
place.
Ideally we would do this in an overload of KConfigSkeletonItem, but
given we can't, I've shadowed the method. This isn't pretty, but given
the docs say it should generally only be used from auto generated code,
should be fine.
TEST PLAN
Used in workspace KCM
REPOSITORY
R237 KConfig
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D25791
AFFECTED FILES
src/core/kcoreconfigskeleton.cpp
src/core/kcoreconfigskeleton.h
To: davidedmundson, ervin
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191206/f8ea2b68/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list