[PATCH]: deleteGroup deletes sub groups too

Oswald Buddenhagen ossi at kde.org
Sun Dec 2 00:43:28 GMT 2007


On Sat, Dec 01, 2007 at 04:58:33PM -0700, Aaron J. Seigo wrote:
> here is a patch to fix deleteGroup such that it deletes not only the keys in 
> that group but also all subgroups.
> 

> --- tests/kconfigtest.cpp	(revision 743218)
> +++ tests/kconfigtest.cpp	(working copy)
> @@ -114,6 +114,16 @@
> +  KConfigGroup ct = cg;
> +  cg = KConfigGroup(&ct, "Nested Group 1");
> +  [...]
>
this looks ... weird.

> --- config/kconfig.cpp	(revision 743218)
> +++ config/kconfig.cpp	(working copy)
>
this is *so* inefficient - the internal api just sucks. whatever.

you might want to update the TODO file.

+1 otherwise.

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Confusion, chaos, panic - my work here is done.




More information about the kde-core-devel mailing list