[Differential] [Commented On] D4175: Introduce a QCommandLineParser for kglobalaccel5

Aleix Pol Gonzalez noreply at phabricator.kde.org
Tue Jan 17 22:49:55 UTC 2017


apol added inline comments.

INLINE COMMENTS

> graesslin wrote in main.cpp:63
> Why did you scope the parser?

So that it frees the QCLP resources as we're not going to be using them anymore. I agree it's not much, but also there's little reason to keep it alive for the whole run of the process.

REPOSITORY
  R268 KGlobalAccel

BRANCH
  master

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: apol, #frameworks, davidedmundson
Cc: graesslin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170117/6cd73b9b/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list