[prev in list] [next in list] [prev in thread] [next in thread]
List: kfm-devel
Subject: Re: some thoughts on libkio
From: Dawit Alemayehu <adawit () earthlink ! net>
Date: 1999-12-07 15:49:32
[Download RAW message or body]
On Tue, 07 Dec 1999, Dawit Alemayehu wrote:
> On Tue, 07 Dec 1999, Stephan Kulow wrote:
> > Dawit Alemayehu wrote:
> > >
> > > On Tue, 07 Dec 1999, Stephan Kulow wrote:
> > > > Dawit Alemayehu wrote:
> > > > >
> > > >
> > > > > > Well, as each app will start a slave when the file dialog uses libkio
> > > > > > for everything, it's basicly a bad idea not to share them.
> > > > >
> > > > > Right. But even with the current design this is not a problem as the io-slaves
> > > > > are shared with KIOSlavePool.
> > > > >
> > > > which does only exist once per app, right? :)
> > >
> > > Nope. That would then not work with multiple instances of konqueror. Someone
> > > did a smart thing when creating KIOSlavePool and added a static member function
> > > that points to the current instance:
> > >
> > > static KIOSlavePool* self();
> > >
> > > This is what what KIOJob uses to create io-slaves:
> > >
> > > KIOSlave* KIOJob::createSlave( const char *_protocol, int& _error, QString& _error_text )
> > > {
> > > KIOSlave *s = KIOSlavePool::self()->slave( _protocol );
> > > ...
> > > }
> > >
> > > Hence, the slaves are actually shared across apps even though each app creates
> > > its own KIOJob.
> > >
> > <joke Do you belief in Santa Claus? /> ;-)
>
> Hehe. I guess so especially since it is Christmas season.
>
> > No, static is only shared between instances of a class within the same
> > address room. Each application using it has it's own copy. If you have
> > several konqueror running, it would have 5 kioslaves each. But if you open
> > several windows of the same konqueror they would have 5 together.
>
> Did not know this.
Well duuhhhhh. I should have known this.
Regards,
Dawit A. (Who is getting a dose if caffine right away!!)
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic