These were intended to suppress GCC warnings about unused variables.
However, this leads to different warnings from Clang instead.
Use another approach that suppresses warnings from both compilers.
Backends use a mixture of SANE_CURRENT_{MAJOR | MINOR} and V_{MAJOR |
MINOR} with all kind of permutations. I was confused by this and one
comment in pieusb.c tells me I was not alone. Some items in old
changelogs suggest to use the SANE_CURRENT_ macros in backends, so let's
switch to do that with the exception of net.c and dll.c. Done with:
$ find backend -name '*.[ch]' | xargs sed -i '/nearly every/ ! { s/\<V_M/SANE_CURRENT_M/g }'
$ git checkout backend/net.c backend/dll.c
And manually removing the comment from pieusb.c. Everything still builds
fine.
The authors have been added to SANE standard itself, pointers to the
standard now point to the published version and/or the project that
isnow used for its maintenance as appropriate. References to input
files and installation locations have been removed.
Removed direct include of usb.h in backend since it
should be using sanei_usb. Completed prototype
of sane_exit(void) for some versions of gcc that are
strict about that.
Signed-off-by: Chris Bagwell <chris@cnpbagwell.com>
David Ward