D25223: Avoid side effects during menu initialization

[Konrad Materka]

kmaterka added a comment.


  In D25223#561638 <https://phabricator.kde.org/D25223#561638>, @nicolasfella wrote:
  
  > My long-term goal is to get rid of the application side KStatusNotifierItem and amend the QSystemTrayIcon API
  
  
  OK, I will think about this. This change is only to fix one regression.
  @broulik Is it OK now? Or should I use std::optional (and require C++17)?

REPOSITORY
  R135 Integration for Qt applications in Plasma

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

To: kmaterka, #plasma, #frameworks, broulik
Cc: nicolasfella, davidedmundson, cgiboudeaux, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191114/e4dd9f05/attachment.html>