On the KonqFrame::attach[Internal]() crash (Analysis, #58480 and others)..

[David Faure]

On Wednesday 21 May 2003 18:07, Waldo Bastian wrote:
> On Thursday 15 May 2003 04:33, Maks Orlovich wrote:
> > Obviously, this should be interrupted someplace along the way, but I
> > don't know where the right spot is. I am willing to do the legwork
> > if someone just wants to direct me on fixing this.
> 
> The following patch fixes it for me.

Idea is ok, but I don't like the fact that the error message (e.g. for undefined symbol
in libkhtmlpart or whatever) isn't shown anymore. It was a convenient way to
find out what's wrong.

> We might want to distinguish here between 
> not finding any suitable factory and encountering load errors when attempting
> to load a factory.

Yup. So why not leave the msgbox - at least when lastErrorMessage isn't empty -,
and leave the isNull check .. if that's enough to fix the problem?

-- 
David FAURE, faure at kde.org, sponsored by TrollTech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
Qtella users - stability patches at http://blackie.dk/~dfaure/qtella.html