bug-groff
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #64463] [groff] subroutine "debug_with_file_and_line" defined diffe


From: G. Branden Robinson
Subject: [bug #64463] [groff] subroutine "debug_with_file_and_line" defined differently in two source files
Date: Mon, 24 Jul 2023 19:09:53 -0400 (EDT)

Update of bug #64463 (project groff):

                  Status:                    None => Need Info              

    _______________________________________________________

Follow-up Comment #2:

[comment #0 original submission:]
> Subject: [groff] subroutine "debug_with_file_and_line" defined
> differently in two source files
> 
>   The files are
> 
> libs/libgroff/error.cpp
>  
>   and
> 
> roff/troff/input.cpp

This is deliberate.  `troff`, the command, has always supplied its own
versions of the diagnostic functions.

https://git.savannah.gnu.org/cgit/groff.git/tree/troff/input.c?h=1.02#n5694

https://git.savannah.gnu.org/cgit/groff.git/tree/lib/error.c?h=1.02#n29

`debug_with_file_and_line` is newer.

https://git.savannah.gnu.org/cgit/groff.git/commit/?id=93ffeffd21868f64af7302cb9574240c8354c60a

>   The subroutine in "roff/trof/input.cpp" is not used there and should
> be removed.

Neither version of `debug_with_file_and_line` should be in the execution trace
of any shipping version of `groff`.  An optimizing linker could (should?)
remove both symbols (along with both versions of `debug`) from each executable
that it generates.  (libgroff.a is not shipped standalone and is only
statically linked.)

>   A compilation of mine showed a linker error which disapered,
> when this subroutine was commented out.

I am interested to see this linker error.  Please provide it.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?64463>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]