Package: deheader
Version: 1.4-1

I discovered this issue when testing deheader on the tsdfx code base.
It claimed log10() required <math.h>, but log10() was not used in the
code.  It was mentioned in a comment, similar to the example below:

  % cat > x.c
  int main(int argc, char *argv[]) { return 0; /* log10(SIZE_MAX) */ }
  % deheader x.c 
  deheader: in x.c, log10() portability requires <math.h>.
  deheader: saw 1 files, 0 includes, 0 removable
  %

As you can see, deheader believe the log10 string is code, while it is
only a comment.  I believe it should ignore the content of comments when
deciding if which header files are required.

-- 
Happy hacking
Petter Reinholdtsen

Reply via email to