[patch] merging/importing kwallets

Michael Leupold lemma at confuego.org
Wed Apr 30 01:27:37 CEST 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. Maybe the patch could be made even smaller by only checking for 
mergedkeys in the final loop but I wasn't quite sure.

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: 2366 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kde-utils-devel/attachments/20080430/04206703/attachment.bin 


More information about the Kde-utils-devel mailing list