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

List:       kde-core-devel
Subject:    Re: Bug in KMainWindow::toolBar()
From:       mosfet <mosfet () mandrakesoft ! com>
Date:       2000-11-27 23:33:40
[Download RAW message or body]

David Faure wrote:
> 
> On Monday 27 November 2000 06:51, mosfet wrote:
> > David Faure wrote:
> > >
> > > On Saturday 25 November 2000 02:19, mosfet wrote:
> > > > Everything seems to be ported to XML by now in kdebase and utils... I
> > > > just tried apps and looked for ones with missing toolbars :P I ran into
> > > > the bug because the thumbnail viewer for Pixie isn't converted to XML
> > > > yet.
> > >
> > > kdelibs/kdeui/tests/kwindowtest.cpp is a plain KMainWindow, no XML,
> > > and the toolbars show up fine.
> >
> > It creates it's own toolbar with new. Remember, I said that works ;-)
> 
> Grr .. please provide an example code (e.g. a diff for kwindowtest)
> that shows the bug. I'm not going to dig around until I see a bug, when
> everything I've tried up to now works just fine.

Grr... you didn't try everything, so what your saying is everything you
use is okay but not everything in the API works :P If you don't want to
fix the bug fine - I already changed my code to work around it. If you
want to see it in kwindowtest simply change the lines that call new
KToolBar to toolBar()! You'll also have to change the variable name that
it uses for the pointer because it uses the name "toolBar", which
obviously conflicts with the method. Stupid stuff.

> 
> > What doesn't is having the mainwindow create it's own automatically by
> > calling toolBar();
> Remind me your nationality ? :-)
> 

WTF does nationality have to do with anything? 


> --
> David FAURE, david@mandrakesoft.com, faure@kde.org
> http://www.mandrakesoft.com/~david/, http://www.konqueror.org/
> KDE, Making The Future of Computing Available Today
> See http://www.kde.org/kde1-and-kde2.html for how to set up KDE 2

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

Configure | About | News | Add a list | Sponsored by KoreLogic