[Libreoffice] [PATCH] Make base debug.hxx free
Jan Holesovsky
kendy at suse.cz
Thu Jul 21 10:32:07 PDT 2011
Hi Jacek,
Jacek Wolszczak píše v So 11. 06. 2011 v 11:21 +0200:
> The patches attached hereto make base debug.hxx free.
> I removed a lot of DBG_CHKTHIS, DBG_CTOR and DBG_DTOR with NULL argument
> and did some standard #ifdef DBG_UTIL to #if OSL_DEBUG_LEVEL > 0
> conversion.
>
> I'm especially interested in getting some feedback on removing functions like:
> DBG_CTOR(ODBTableDecorator, NULL);
> DBG_DTOR(ODBTableDecorator, NULL);
Terribly sorry for the late answer, your mail fell through the cracks
somehow :-(
So - I myself have never used this kind of debug, and I think, it is
much better to use valgrind to check for bugs that might be somehow
uncovered by these DBG_CTOR etc. checks (if I understood them
correctly).
Though, they seem to be used more widely than just in base; so my
preferred approach (if we want to get rid of these) would be to come up
with a script that will remove the DBG_CTOR / DBG_DTOR / ... over all
the codebase - how does that sound?
Thank you,
Kendy
More information about the LibreOffice
mailing list