KPluginMetaData: use Q_DECLARE_METATYPE
ClosedPublic

Authored by apol on Jul 4 2019, 1:02 PM.

Details

Summary

This way it can be used with QVariant.

Diff Detail

Repository
R244 KCoreAddons
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 13596
Build 13614: arc lint + arc unit
apol created this revision.Jul 4 2019, 1:02 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 4 2019, 1:02 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
apol requested review of this revision.Jul 4 2019, 1:02 PM
mart accepted this revision.Jul 4 2019, 1:43 PM
This revision is now accepted and ready to land.Jul 4 2019, 1:43 PM
This revision was automatically updated to reflect the committed changes.

This runs the chance to break some 3rd-party software which also calls the Q_DECLARE_METATYPE(KPluginMetaData) macro.
Any chance this could be moved to the place that needs this for now, and perhaps add a KF6 TODO instead?

See e.g. R36:3aacbbaab50ac6e0557c5d69c430459eb3d71ad7 which was needed to unbreak ark builds.

This runs the chance to break some 3rd-party software which also calls the Q_DECLARE_METATYPE(KPluginMetaData) macro.
Any chance this could be moved to the place that needs this for now, and perhaps add a KF6 TODO instead?

See e.g. R36:3aacbbaab50ac6e0557c5d69c430459eb3d71ad7 which was needed to unbreak ark builds.

Ping?