[PATCH] Revised patch for KUser
Harri Porten
porten at froglogic.com
Mon Jun 7 22:04:20 BST 2004
On Mon, 7 Jun 2004, Jan [iso-8859-15] Schäfer wrote:
> I have revised my changes according to your comments.
> The new patch is attached.
I see no reference counting and delete statement. At first sight it looks
like 'd' will be leaking.
A optimization suggestion: the temporary allGroups and users variables
QValueList<KUserGroup> allGroups = KUserGroup::allGroups();
QValueList<KUser> users = group.users();
can probably be declared const to avoid detaching.
Harri.
More information about the kde-core-devel
mailing list