On Fri, Jul 07, 2006 at 07:13:42PM +0100, seventh guardian wrote:
> After some checking around, it seems that the file libs/debug.c isn't
> used anymore. The file was created in 1998 as a debuging library, but
> it seems to have been replaced by simpler solutions.

I think Paul Smith put it there for future use back then, but we
never started using that library.

> It's part of libfvwm.a, and the *.c code is surrounded by a #ifdef
> DEBUG, so never gets compiled. The whole file is "empty" at the
> moment, except for an int variable that is there (outside the #ifdef)
> expressly to prevent an empty file, as some compilers don't like that
> (it's written on the coments). On the other hand, the header file
> fvwmlib.h has its macros and structs allways defined, (but there is no
> compiled code!). Nothing is ever used, so I think it's safe to remove
> it.

Yes, it's definitely safe.

> These are the only places where you find the macros (which need to be 
> removed):
> - DB() is in a #if 0 part of module_interface.c (but the rest of the
> file already uses the DBUG macro) and in focus.
> - __FILE__ and __LINE__ is in a ifdef'ed to 0 part of focus.h where
> alternative debuging macros are defined.
> 
> It's a big decision to remove a whole file, so I need a second
> opinion.. Is the file still useful or may I remove it?

Well, usefull or not - if we don't use it we don't need it.

Ciao

Dominik ^_^  ^_^

 --
Dominik Vogt, [EMAIL PROTECTED]

Attachment: signature.asc
Description: Digital signature

Reply via email to