[PATCH] kwalletd, get rid of delayed messages

Michael Leupold lemma at confuego.org
Sat Sep 20 10:11:35 BST 2008


Hi,

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)

Benefits:
- no more problems with timeouts on open-calls (currently I just set a huge 
timeout to not make that happen in 4.1)
- we get rid of the QDBusMessage arguments in the adaptor and thus can auto-
generate it (currently there's a redundancy between the interface.xml in 
kdelibs and the adaptor.h in kdebase).

I'm in process of adapting KWallet::Wallet to use the new mechanism. To keep 
the DBus interface compatible I reimpleneted the KWalletD::open call to use 
the new openAsync call + an eventloop.

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


More information about the kde-core-devel mailing list