D28714: Refactor enableBluetooth function
Nicolas Fella
noreply at phabricator.kde.org
Fri Apr 10 01:28:10 BST 2020
nicolasfella created this revision.
nicolasfella added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
Herald added a reviewer: jgrulich.
nicolasfella requested review of this revision.
REVISION SUMMARY
The code has quite some boilerplate and high levels of indentation, resulting in poor readability. Factoring out the DBus boilerplate makes it easier to focus on the essential parts.
TEST PLAN
(de)activated flight mode. Bluetooth was (dis)abled accordingly
REPOSITORY
R116 Plasma Network Management Applet
BRANCH
bluetooth
REVISION DETAIL
https://phabricator.kde.org/D28714
AFFECTED FILES
libs/handler.cpp
To: nicolasfella, #plasma, jgrulich
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200410/8f9a2642/attachment.html>
More information about the Plasma-devel
mailing list