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

List:       kde-devel
Subject:    Re: kDebugInfo definition problem
From:       David Faure <faure () kde ! org>
Date:       2000-02-04 18:17:56
[Download RAW message or body]

On Thu, Feb 03, 2000 at 06:36:46PM +0000, Stephan Kulow wrote:
> Yves Arrouye wrote:
> > 
> > Hi,
> > 
> > A problem happens when one compiles kdelibs normally, but compiles kdebase
> > with --enable-debug. The reason for that is that kdelib will not define the
> > kDebugInfo() functions since it has NDEBUG defined, but kdebase will call
> > kDebugInfo and link will fail.
> > 
> > It is quite easy to correct by always providing a definition for
> > kDebugInfo() in the .cpp, but I have a question about the following snippet
> > of code in kDebugBackend:
> > 
> > #ifdef NDEBUG
> >         delete pConfig;
> >         pConfig = 0;
> > #endif
> > 
> > How should this be changed in order to accomodate the situation I'm into,
> > which is kdelibs compiled normally and kdebase compiled with --enable-debug?
> > Or is this a situation that KDE won't support (after all, I can also compile
> > kdelibs w/ --enable-debug and then I'll be able to debug everything; but
> > then too I will have all these spurious info messages I don't really care
> > about).
> > 
> Hmm, the pasted code makes kDebugInfo faster, but requires more memory.
> I don't see a way to support that, so I'd vote for not supported.

So ? Does this mean the (somewhat hacky) commit should be reverted ?

> Please note that giving no arguments to configure will compile without
> NDEBUG,
> just --disable-debug gives that - so if you do that in kdelibs, it
> should be quite
> obvious that you HATE debugging ;-)
> 
> You can turn out almost all debug outputs using kdebugdialog

Yup.
I see the feature "apply this for all areas" being asked soon :-)

-- 
David FAURE
david@mandrakesoft.com, faure@kde.org
http://home.clara.net/faure/
KDE, Making The Future of Computing Available Today

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

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