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

List:       kde-devel
Subject:    Re: KWrite and KParts
From:       Michael Koch <m_koch () bigfoot ! de>
Date:       2000-05-11 6:15:41
[Download RAW message or body]

Am Mit, 10 Mai 2000 schrieb David Faure:
> On Thu, May 11, 2000 at 12:31:57AM +0400, Alexei Dets wrote:
> > Hi!
> > As I notice KWrite is a KPart now.
> > I don't need KWrite as a whole thing, I need slightly stripped down
> > version.
> > And I need to inherit it because I'm pretty sure that I'll need to
> > change KWrite_Doc and I can't simply inherit from KWrite_Doc because it
> > is a friend class but its child will not be.
> > So, I must inherit KWrite, KWrite_Doc and KWrite_View - they are all
> > "friends" :-(
> > I can say that my KWrite will not be KWrite from kdelibs, why I need
> > this strange and useless in my case KPart thing?
> > So, the question is: what is the fastest way to get rid of this bloat
> > and convert KWrite back to plain QWidget?

The current what the KPart is defined is not good IMO. I saw that this mail
would come when Jochen commited this.

> Why ? A part is nothing else than a widget and some actions
> that go with it. You probably want the actions too.
> No bloat here.
> 
> (I agree that a design where all classes are friend is not a good
> design, though).
> 
> Beware that the API of libkwrite will most probably change before the 
> final 2.0. For now it's in kdelibs only to make the part available...

yep, I'm working on removing the "friend" declarations. they are horrible.
Some apps that want to embed KWrite does not need the KPart-part,
e.g. KSql ...... but thats another story.I will speak to Jochen about his
problem.


Ciao,
Michael
--
koch@kde.org, m_koch@bigfoot.de
http://heaven.riednet.wh.tu-darmstadt.de/~mkoch

To me vi is Zen.  To use vi is to practice zen. Every command is
a koan. Profound to the user, unintelligible to the uninitiated.
You discover truth everytime you use it.
                                 -- reddy@lion.austin.ibm.com

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

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