[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