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