[Differential] [Commented On] D3088: [KDecoration] Restore application menu button

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> graesslin wrote in decoration.cpp:201
> I agree, static_cast doesn't make sense. I would do a dynamic_cast/qobject_cast and check for null.

DecoratedClientPrivate is not a QObject, so only option is dynamic_cast

REPOSITORY
  R129 Window Decoration Library

REVISION DETAIL
  https://phabricator.kde.org/D3088

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: davidedmundson, sitter, graesslin, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, andreaska, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170110/5a2d5798/attachment.html>