D14493: fontinst quits after KJob is done

[David Faure]

dfaure added a comment.


  Ah so it's kind of a daemon, that should keep running until logout? I wasn't aware of that, I thought it was a KCM. Disabling as in the first patch made sense then ;)
  
  Either version is fine with me.

INLINE COMMENTS

> FontInst.h:161
> +    // This is needed because KJob uses an event loop locker internally,
> +    // which causes fontinst to quit after the job is done.
> +    QEventLoopLocker loopLocker;

"which would otherwise cause fontinst...."

Don't describe the past as the present, for future readers of this code ;)

REPOSITORY
  R119 Plasma Desktop

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

To: mathiastillman, #plasma, dfaure, aacid, gateau
Cc: wbauer, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180731/9e025e65/attachment.html>