The rationale is explained in https://planet.kde.org/friedrich-kossebau-2023-06-28-include-also-moc-files-of-headers/
In case of KDEConnect, it impressively speeds up compilation. Before it
took 390 seconds on a clean build and with this change it took 330 seconds.
This is due to the mocs_compilation having to include the header files
and thus all their headers. Due to the lots of small plugins we have,
this means that the same headers must be compiled plenty of times.
When we include the moc files directly in the C++ file, they are already
available.
In theory we support Qt 5.6 (because of SailfishOS) but in practice we are not checking that in the CI so probably we broke the compatibility long ago. Also, I don't think anyone is using this code in SailfishOS, so we can greatly simplify the code by requiring Qt 5.15 or later.
Summary: It's not only used by the plasmoid any more but also SFOS and the app use it.
Test Plan: Builds, App runs fine
Reviewers: #kde_connect, piggz, apol
Reviewed By: #kde_connect, piggz, apol
Subscribers: piggz, kdeconnect
Tags: #kde_connect
Differential Revision: https://phabricator.kde.org/D14567
2018-08-03 02:24:58 +02:00
Renamed from plasmoid/declarativeplugin/responsewaiter.cpp (Browse further)
