[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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic