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

List:       kde-devel
Subject:    Re: From qDebug() to kdDebug()
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-05-31 13:38:32
[Download RAW message or body]

On Thursday 31 May 2001 15:56, Gioele Barabucci wrote:
> On Thursday 31 May 2001 13:33, David Faure wrote:
> >
> > And
> > -      qDebug("KLauncher: Fatal error, can't create tempfile!");
> > +      kdDebug(7016) << "KLauncher: Fatal error, can't create tempfile!" <<
> > endl;
> >
> >        ::exit(1);
> >
> > is wrong. We want fatal errors to always appear. Don't forget that kdDebug
> > is disabled by NDEBUG....
> kdFatal() ?

kdFatal calls abort(), so we end up with a core dump.
Not wanted here, when we know the cause for abortiong.
kdError() is what you want for such cases.
But in KLauncher, (which is quite low-level), I'd avoid doing that.
I'm not even sure it has a kinstance around.
There's nothing wrong with the above qDebug, it's _not_ debug output.

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

 
>> Visit http://master.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

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