[Kde-pim] Review Request 123773: Port KontactInterface away from KUniqueApplication (and KDELibs4Support)

Daniel Vrátil dvratil at kde.org
Thu May 14 13:14:09 BST 2015



> On May 14, 2015, 2:07 p.m., Laurent Montel wrote:
> > src/uniqueapphandler.cpp, line 96
> > <https://git.reviewboard.kde.org/r/123773/diff/1/?file=368724#file368724line96>
> >
> >     Do you want to reactivate it ?

Yes: the entire point of qCDebug() is not to have to comment/uncomment debug messages in the code all the time. Instead we can just disable kontactinterface_log.debug *by default* and only turn it on via your tool, or via cmdline, right?


> On May 14, 2015, 2:07 p.m., Laurent Montel wrote:
> > src/uniqueapphandler.h, line 44
> > <https://git.reviewboard.kde.org/r/123773/diff/1/?file=368723#file368723line44>
> >
> >     You changed it.
> >     Did you changed
> >     "./mailcommon/util/mailutil.cpp:                             QLatin1String("org.kde.KUniqueApplication"));
> >     ./mailcommon/util/mailutil.cpp:                                    "org.kde.KUniqueApplication");
> >     ./kmail/kmkernel.cpp:                         QLatin1String("org.kde.KUniqueApplication"),
> >     ./kalarm/functions.cpp:    QDBusInterface iface(KORG_DBUS_SERVICE, QLatin1String(KORG_DBUS_LOAD_PATH), QLatin1String("org.kde.KUniqueApplication"));
> >     " ?

Oops :) I'll fix that in the other review.


- Daniel


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123773/#review80343
-----------------------------------------------------------


On May 13, 2015, 3:54 p.m., Daniel Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123773/
> -----------------------------------------------------------
> 
> (Updated May 13, 2015, 3:54 p.m.)
> 
> 
> Review request for KDEPIM-Libraries and Laurent Montel.
> 
> 
> Repository: kontactinterface
> 
> 
> Description
> -------
> 
> Port away from KUniqueApplication, use QApplication + manual DBus invocation instead (we can't use KDBusService either because it's not customizable enough for our needs). Basically the only change is renaming newInstance() to activate() (because we want to activate the running instance, right?) and passing list of arguments, because we no longer have the global static KCmdArgs and calling QApplication::arguments() is expensive (according to documentatation).
> 
> Another reason for renaming newIntance() to activate() is that I'm planning to use this call from individual application's systray icon implementations, which will allow us to automatically switch to the correct module in Kontact (i.e. right now when have "Calendar" active in Kontact, clicking on KMail systray icon restores Kontact window, but does not switch to KMail module - this will be possible with calling activate()).
> 
> 
> See dev/no-kuniqueapplication branch for commits.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 176c47f6d50e7a13daa5113f9574dde3528ce29e 
>   src/CMakeLists.txt a44afed97f8a83bbeaab8d0fc1e807c5d03cb2ac 
>   src/core.h 67837a93b481fcccf90aba04202580741cf68b83 
>   src/core.cpp 03afef28b27cb004ec92751e007792ee41133b0b 
>   src/pimuniqueapplication.h 875d561abeb3850593ba40476d2cb0c4c5bc0ad0 
>   src/pimuniqueapplication.cpp bbe4218c97dbc7ee2e38b2007d006ffd575d2624 
>   src/plugin.h 57f621ba7a47e1a852c2f5c12c3b67454a7ad520 
>   src/plugin.cpp bb1422e6b3323fd07aed56ee24e4f91621b11d21 
>   src/summary.cpp 15f123188ba14da4c7ad594bfdbb2b40f3928296 
>   src/uniqueapphandler.h 08185e5b073d8685f65cd4de7c23a7279c064e31 
>   src/uniqueapphandler.cpp 5fcf9cd859d4c9625d3cd16b76ead14c02f7d862 
> 
> Diff: https://git.reviewboard.kde.org/r/123773/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Daniel Vrátil
> 
>

_______________________________________________
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/


More information about the kde-pim mailing list