[PATCH] Revised patch for KUser

Simon Hausmann hausmann at kde.org
Mon Jun 7 23:00:02 BST 2004


On Monday 07 June 2004 23:08, André Wöbbeking wrote:
> On Monday 07 June 2004 23:04, Harri Porten wrote:
> > 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.
>
> I think you mean const&.

No, as allGroups() and users() return values, and references to temporaries 
are not a good idea :)

Simon




More information about the kde-core-devel mailing list