On sábado, 17 de novembro de 2012 10.55.01, David Faure wrote:
> My current broken patch is
> http://www.davidfaure.fr/2012/kuniqueapp_broken_fix.diff
>
> The change from QDBusConnectionInterface* to QDBusConnection as the return
> value of tryToInitDBusConnection() -- which introduces a bit of noise in
> the  patch -- is only due to the very last hunk: so that we can make the
> call from parent to child using the private connection rather than
> sessionBus().

The part in kuniqueapplication.cpp is evident, but I haven't got a clue what
you're trying to do in kapplication.cpp. You'll need to explain.

The rest looks ok.

Looking at the code today I see many uses of ::exit in both sides of the fork,
which is bad. This should be cleaned up, but since it's been like that for
ages, let's just leave it.

--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   Software Architect - Intel Open Source Technology Center
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to