dfaure added a comment.

  Looks simpler indeed.

INLINE COMMENTS

> engine.cpp:58
> +typedef QHash<QString, XmlLoader*> EngineProviderLoaderHash;
> +Q_GLOBAL_STATIC(QThreadStorage<EngineProviderLoaderHash>, 
> EngineProviderLoaders)
> +

The uppercase first letter on variable name is unusual.

I personally use a s_ prefix for static vars.

REPOSITORY
  R304 KNewStuff

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

To: leinir, whiting, apol
Cc: dfaure, #frameworks

Reply via email to