This revision was automatically updated to reflect the committed changes.
Closed by commit R237:9dec6961b3ae: [KConfigLoader] Code cleanup (authored by
ahmadsamir).
REPOSITORY
R237 KConfig
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D27157?vs=75029&id=75210
REVISION DETAIL
https
dfaure accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R237 KConfig
BRANCH
l-configloader (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D27157
To: ahmadsamir, #frameworks, apol, dfaure
Cc: kde-frameworks-devel, LeGast00n, cblack, GB
ahmadsamir updated this revision to Diff 75029.
ahmadsamir added a comment.
Tweak commit message
REPOSITORY
R237 KConfig
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D27157?vs=75008&id=75029
BRANCH
l-configloader (branched from master)
REVISION DETAIL
https://phabricator.kd
ahmadsamir created this revision.
ahmadsamir added reviewers: Frameworks, apol, dfaure.
Herald added a project: Frameworks.
ahmadsamir requested review of this revision.
REVISION SUMMARY
- Drop redundant args to startElement()/endEelement()
- Use range-for
- Drop ConfigLoaderHandler methods
