Konqy reusing and preloading

[Lubos Lunak]

On Friday 29 of November 2002 14:37, Lubos Lunak wrote:
> On Thursday 28 of November 2002 14:53, Carsten Pfeiffer wrote:
> > -----BEGIN PGP SIGNED MESSAGE-----
> >
> > On Thursday 28 November 2002 13:25, David Faure wrote:
> > > >  The reuse.patch one is for #27583. (Bonus points to whoever wrote
> > > > startNewKonqueror() for confusing me twice before I got it right.) I
> > > > hope adding new and new methods in KonquerorIface doesn't matter
> > > > much.
> >
> > I was confused by it as well, recently :)
> >
> > > Great, a solution that uses findObject() - it's exactly what I had in
> > > mind in my discussion with Carsten.
> >
> > Weeellllll, but this is still the "local" vs. "remote" option, which
> > doesn't make much sense, IMHO. When I'm browsing an ftp-site or sftp,
> > audiocd or something like that, I still want it to reuse an existing
> > process.
> >
> > IMHO this is the same as kicker's "[un]trusted applets" configuration.
> > You trust e.g. KonqIconView to not crash, independently of the URL it is
> > currently showing.
> >
> > I wouldn't trust khtml tho, be it a local file or a remote one. The
> > source of the data doesn't make konqueror more or less crash-prone, the
> > data itself (and the component to view it) does.
>
>  This is true. Not that it makes that much difference though. If you reuse
> Konqy with icon view for another icon view, you can later switch to showing
> HTML in one of the windows, so you'll be reusing KHTML. The same problem is
> of course with local vs remote, but there's it's IMHO less likely it will
> change.
>
>  But it shouldn't be that difficult to change the two DCOP function I added
> to check view types instead.

 Ok, another try, for #27583 only, there's some #ifdef KONQY_PRELOADING code, 
I'll try to make that reuse the last KonqMainWindow. There's now a config 
entry for parts that are considered safe and such Konqy processes may be 
reused (the problem with changing the part later remains of course). By 
default the allowed ones are KonqKfmIconView and KonqListView, should there 
be more?

 This would also require changing the kcontrol module. No code yet, I'm not 
sure how to do it. Listing parts doesn't IMHO make much sense. Maybe 'never 
reuse'/'always reuse'/'safe reuse' , where 'safe reuse' will means the two 
parts listed above (and maybe more, I don't know all the parts, and I'm not 
sure if parts like Kate should be there).

 Comments on this one? The improved preloading patch will come later.

-- 
Lubos Lunak
KDE developer
---------------------------------------------------------------------
SuSE CR, s.r.o.  e-mail: l.lunak at suse.cz , l.lunak at kde.org
Drahobejlova 27  tel: +420 2 9654 2373
190 00 Praha 9   fax: +420 2 9654 2374
Czech Republic   http://www.suse.cz/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: KonquerorIface.cc.patch
Type: text/x-diff
Size: 2896 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20021129/ce84adc2/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: KonquerorIface.h.patch
Type: text/x-diff
Size: 604 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20021129/ce84adc2/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kfmclient.cc.patch
Type: text/x-diff
Size: 4800 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20021129/ce84adc2/attachment-0002.patch>