[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 11:33:24
[Download RAW message or body]

On Thursday 31 May 2001 14:49, Gioele Barabucci wrote:
> On Wednesday 30 May 2001 20:43, David Faure wrote:
> > On Wednesday 30 May 2001 19:19, Gioele Barabucci wrote:
> > > Bored from noisy qDebug() calls I have made moved to kdDebug() some files
> > > from kdelibs and kdebase.

Hmm, you know, anything that's inside #ifdef SOMETHING_DEBUG
is something that gets never printed, unless the developer enables it
for himself. Not really worth converting.

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....

I applied css and render_hr, the rest of the kdelibs patch doesn't look useful.

-- 
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] 

Configure | About | News | Add a list | Sponsored by KoreLogic