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

List:       kfm-devel
Subject:    Re: PATCH adds PluginInterface to KonqPopupMenu
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-10-30 13:21:59
[Download RAW message or body]

On Mardi 30 Octobre 2001 14:17, Holger Freyther wrote:
-- > On Tuesday 30 October 2001 14:12, David Faure wrote:
> > On Mardi 30 Octobre 2001 13:41, Simon Hausmann wrote:
> > > Hm, ok, we already have popup( const QPoint & ) , so I guess another
> > > stub doesn't hurt. Hmmm, I agree on an insert( KActionMenu * ) stub
> > > (David, what do you think?)
> >
> > KActionMenu already has insert(KAction *). And KActionMenu is an action
> > so in terms of API it doesn't make much sense to have both.
> >
> > Doesn't insert(KAction*) work for kactionmenus? It calls plug, which
> > inserts the whole popupmenu, for action menus, doesn't it ?
> >
> > -> I don't see the need for this, it seems to me that insert(KAction *)
> > does it all already.
> >
> > David.
> read some posts before. Yes it should work with a down_cast but it would work 
> Simons way but I thought it was cleaner this was not eeding to do this cast 
> in an application

There is no need for a cast when passing a KActionMenu * to a method
that takes a KAction * parameter.

--
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/ , http://www.konqueror.org/
KDE 3.0: Konquering the Desktops

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