[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #64463] [groff] subroutine "debug_with_file_and_line" defined diffe
From: |
Dave |
Subject: |
[bug #64463] [groff] subroutine "debug_with_file_and_line" defined differently in two source files |
Date: |
Thu, 27 Jul 2023 19:36:12 -0400 (EDT) |
Follow-up Comment #5, bug #64463 (project groff):
[comment #2 comment #2:]
> `debug_with_file_and_line` is newer.
>
>
https://git.savannah.gnu.org/cgit/groff.git/commit/?id=93ffeffd21868f64af7302cb9574240c8354c60a
If anything, I think the significant thing here is that the info "There are no
call sites for debug() [and debug_with_file_and_line()] and I don't plan for
there to be; I am adding these for developer convenience" appears in the above
commit message but in no code comments. The name debug*() does hint at this
aspect, but in general it's worthwhile to comment intentional dead code so
that later code audits don't remove it.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?64463>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/