D28591: Add #ifndef QT_NO_SYSTEMTRAYICON to KleopatraApplication::~Private()

Andrey Legayev noreply at phabricator.kde.org
Sun Apr 5 13:52:38 BST 2020


andreylegayev created this revision.
andreylegayev added a reviewer: mlaurent.
andreylegayev added a project: KDE PIM: Kleopatra.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
andreylegayev requested review of this revision.

REVISION SUMMARY
  I've found that class member `sysTray` is not created if QT_NO_SYSTEMTRAYICON defined: see it in kleopatraapplication.cpp line 148 <https://phabricator.kde.org/source/kleopatra/browse/master/src/kleopatraapplication.cpp$148>
  I've added the same pre-processing macros for delete statement to prevent compilation error.

TEST PLAN
  Try to compile with QT_NO_SYSTEMTRAYICON - it should proceed without errors.

REPOSITORY
  R168 Kleopatra

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

AFFECTED FILES
  src/kleopatraapplication.cpp

To: andreylegayev, mlaurent
Cc: kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, ach, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200405/b9690608/attachment-0001.html>


More information about the kde-pim mailing list