D24466: Use ECMGenerateExportHeader to manage deprecated API better
David Faure
noreply at phabricator.kde.org
Sat Oct 12 20:44:50 BST 2019
dfaure added inline comments.
INLINE COMMENTS
> kactioncollection.h:278
> */
> + KXMLGUI_DEPRECATED_VERSION(5, 0, "Do not rely on")
> QT_MOC_COMPAT void removed(QAction *action);
Does this work? It's a signal. Won't this only warn at emit time? (while QT_MOC_COMPAT is what leads to a warning at connect time)
Asking because you didn't do this in D24465 <https://phabricator.kde.org/D24465> for the windowChanged() signals, so something seems inconsistent.
> kactioncollection.h:314
> + KXMLGUI_DEPRECATED_VERSION(5, 0, "Use KActionCollection::slotActionHovered()")
> QT_MOC_COMPAT virtual void slotActionHighlighted();
> +#endif
(pre-existing) QT_MOC_COMPAT for a slot? Any idea what this might do?
REPOSITORY
R263 KXmlGui
BRANCH
deprecatedapi
REVISION DETAIL
https://phabricator.kde.org/D24466
To: kossebau, #frameworks, dfaure, mlaurent
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191012/3cc1a45c/attachment.html>
More information about the Kde-frameworks-devel
mailing list