Review Request 117161: Drop QApplication usages in units.cpp

Aleix Pol Gonzalez aleixpol at kde.org
Sat Mar 29 19:25:34 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117161/
-----------------------------------------------------------

Review request for KDE Frameworks, Marco Martin and Sebastian Kügler.


Repository: plasma-framework


Description
-------

Drop dependency to QtWidgets from this file. We can start assuming that it might not be that functional in some platforms.

Use QGuiApplication counterparts, based mostly on QScreen, which could make it more powerful in the future.

I did it because otherwise KAlgebraMobile crashes if using the Plasma Components based interface.


Diffs
-----

  src/declarativeimports/core/units.cpp f0b8d39 

Diff: https://git.reviewboard.kde.org/r/117161/diff/


Testing
-------

Builds, KAlgebra doesn't have the problem now.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140329/ff569b8d/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list