[PATCH] kwalletd, get rid of delayed messages

Michael Leupold lemma at confuego.org
Sat Sep 20 18:53:53 BST 2008


On Saturday 20 September 2008, Thiago Macieira wrote:
> Michael Leupold wrote:
> >following a suggestion by Thiago I propose the attached patch to
> > kwalletd. It gets rid of delaying messages by changing how the a wallet
> > SHOULD be opened: - client sends an openAsync methodcall
> >- server returns a transaction identifier and starts handling the
> > transaction - client listens to the server's walletAsyncOpened signal
> > (which will be emitted on success and failure)
> Sounds like the best solution.
> Thanks for doing that.

I found one major bug that made the daemon crash on cancelling entering the 
password. The client code is ready as well now.

One question about binary compatibility:
Besides adding two private slots I also removed two which are no longer 
needed. That should be ok, shouldn't it?

Ok to commit?

Regards,
Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kwalletd-asyncopen-2.diff
Type: text/x-patch
Size: 13115 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080920/cb2812f3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kwallet-kdelibs-asyncopen.diff
Type: text/x-patch
Size: 6289 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080920/cb2812f3/attachment-0001.bin>


More information about the kde-core-devel mailing list