D28067: check for window existence

Arjen Hiemstra noreply at phabricator.kde.org
Mon Mar 16 09:58:41 GMT 2020


ahiemstra accepted this revision.
ahiemstra added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> shadowedrectangle.cpp:178
> +
> +void ShadowedRectangle::checkSoftwareItem()
> +{

Just a minor nitpick: Please move this to the end of the file so that the method order in the source matches that of the header.

REPOSITORY
  R169 Kirigami

BRANCH
  phab/nocrash

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

To: mart, #kirigami, ahiemstra
Cc: ahiemstra, plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, davidedmundson, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200316/fec63f01/attachment.html>


More information about the Plasma-devel mailing list