apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.


  Yes, it's a big one.
  
  I was wondering if it would make sense to see if we can improve it on Qt 
level.

INLINE COMMENTS

> DiscoverNotifier.cpp:166
>  {
> +    if (!m_manager) {
> +        m_manager = new QNetworkConfigurationManager(this);

Explain why in a short comment. Something like `// delay initializing, Qt does 
a bunch of blocking calls there when using networkmanager`

REPOSITORY
  R134 Discover Software Store

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

To: broulik, #plasma, apol
Cc: plasma-devel, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to