Differential D26870

[ksgrd] Drop a QWidget dependency
ClosedPublic
Actions

Authored by davidedmundson on Jan 23 2020, 2:46 PM.

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R111:d7a5f72c7a69: [ksgrd] Drop a QWidget dependency
Summary

The broadcaster just needs to receive a QEvent getting propogated
upwards, it doesn't need to be a QWidget.

This makes the client API considerably lighter.

.so is bumped as it is technically an ABI break.

Test Plan

Ksysguard still works
lddtree shows everything being much lighter

Diff Detail

Repository
R111 KSysguard Library
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21578
Build 21596: arc lint + arc unit
davidedmundson created this revision.Jan 23 2020, 2:46 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 23 2020, 2:46 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Jan 23 2020, 2:46 PM
apol accepted this revision.Jan 23 2020, 2:58 PM
This revision is now accepted and ready to land.Jan 23 2020, 2:58 PM
Closed by commit R111:d7a5f72c7a69: [ksgrd] Drop a QWidget dependency (authored by davidedmundson). · Explain WhyJan 24 2020, 11:18 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
MCMakeLists.txt (2 lines)
Mksgrd/CMakeLists.txt (2 lines)
Mksgrd/SensorManager.h (4 lines)
Mksgrd/SensorManager.cpp (5 lines)

Diff 74238

View Options

CMakeLists.txt

Loading...
View Options

ksgrd/CMakeLists.txt

Loading...
View Options

ksgrd/SensorManager.h

Loading...
View Options

ksgrd/SensorManager.cpp

Loading...