[Kde-pim] Akonadi Commandline Interface Project

[Kevin Krammer]

On Monday, 2014-03-03, 02:24:10, Bhaskar Kandiyal wrote:
> Hi!
> 
> I've made a few changes to the application [1], mainly:
> 
> * Moved QCoreApplication initialization to main() - Should I send a
> patch for review?

I would suggest to also rename CommandRunner::exec() to 
CommandRunner::start(), it no longer has exec() behavior.

Also the main idea of the move was to create the application object before the 
runner.

Almost everything in the command runner constructor could be moved to start.
Maybe have a CommandFactory member and initialize it, but creation of command 
and init can be done in start(), so start can return error whenever it needs, 
without having to call exit itself.

Cheers,
Kevin
-- 
Kevin Krammer, KDE developer, xdg-utils developer
KDE user support, developer mentoring
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20140302/c8bb15e5/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/