[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