a2ps-4.14 bug report

[David Binderman]

Hello there,

I recently tried out the cppcheck static analysis tool on the source
code of a2ps-4.14

It said

[a2ps-4.14/lib/signame.c:306]: (warning) Mutual exclusion over || always evaluates to true. Did you intend to use && instead?

The source code is

  if (signal > 0 || signal < NSIG)

More plausible code might be

  if (signal > 0 && signal < NSIG)

Regards

David Binderman