[PATCH][132808] kwalletmanager: fix merging kwallets

Michael Leupold lemma at confuego.org
Thu May 1 08:54:34 BST 2008


Hi,

this patch aims at fixing http://bugs.kde.org/show_bug.cgi?id=132808. The 
problem arises basically because kwallet only allows to specifically query 
for maps and passwords. You get stream and unused entries using a generic 
query that returns all data (without being able to figure out the type after 
retrieval). I made kwalleteditor.cpp remember which entries it already 
merged using a QSet.

IMO the "right thing" to do would be to add functions for streams and unused 
(whatever that's supposed to be used for) entries as well. Wouldn't even 
break the api for existing apps using the interface :) Is kwalletd managed by 
the kdelibs maintainers?

Regards,
Lemma
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch-kwallet-importwallet.diff
Type: text/x-diff
Size: 1468 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080501/29deeeb2/attachment.diff>


More information about the kde-core-devel mailing list