D19976: [about-distro] OSRelease to collect "extra" keys
Harald Sitter
noreply at phabricator.kde.org
Fri Mar 22 14:00:31 GMT 2019
sitter created this revision.
sitter added a reviewer: apol.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.
REVISION SUMMARY
previously we'd ignore unknown keys. in preparation of moving to a library
let's instead collect them so a user of the class may also access them.
- new extraKeys helper and extraValue getters
- extras are always processed as String
- test case fixture extended + now has comment variants (previously comments couldn't be tested since they were skipped as either comments or invalid/unknown lines. with the extras API we can actually verify that they do not incorrectly end up getting parsed
TEST PLAN
test still passes
REPOSITORY
R102 KInfoCenter
BRANCH
extras
REVISION DETAIL
https://phabricator.kde.org/D19976
AFFECTED FILES
Modules/about-distro/autotests/OSReleaseTest.cpp
Modules/about-distro/autotests/data/os-release
Modules/about-distro/src/OSRelease.cpp
Modules/about-distro/src/OSRelease.h
To: sitter, apol
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190322/d3eace97/attachment-0001.html>
More information about the Plasma-devel
mailing list