vonreth added inline comments.

INLINE COMMENTS

> pino wrote in notifybysnore.cpp:92
> please document what is the issue, so in the future the workaround can be 
> dropped

I can't put my finger on it I can only tell that 
https://github.com/KDE/snoretoast/blob/a53a6b733624ada78ef0f852407cad4be1c00b16/examples/qt/main.cpp#L58
 produces a corrupted string.
Runs fine with msvc2017.
I tried to create a minimal example https://invent.kde.org/snippets/253 but 
this one works fine..

REPOSITORY
  R289 KNotifications

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

To: brute4s99, broulik, sredman, vonreth, albertvaka
Cc: nicolasfella, pino, kde-frameworks-devel, LeGast00n, michaelh, ngraham, 
bruns

Reply via email to