[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: patch: kaction and kghostview
From:       Wilco Greven <greven () kde ! org>
Date:       2002-01-29 14:16:41
[Download RAW message or body]

On Tuesday 29 January 2002 00:19, Simon Hausmann wrote:
> Hi,
>
> The KActionPtrList class is a recent addition to
> KActionCollection. After a discussion with Ellis it turned out this
> class adds no real value over a plain QValueList<KAction *> , which
> is also what KDE 2's KActionCollection used to have in the API.

I don't think KActionPtrList is a good name. It suggests that it is a 
QPtrList instead of a QValueList. How about KActionList, or a typedef List 
inside the KAction declaration so that you get a KAction::List.

Greetings,
Wilco
[prev in list] [next in list] [prev in thread] [next in thread]