On Tue, Jan 6, 2015 at 12:00 PM, Linus Torvalds torvalds@linux-foundation.org wrote:
Why didn't I notice the compiler warning? Hmm. Probably because the build system is so verbose. There is a reason why the kernel (and subsurface) builds try to be quiet. Or maybe the warning flags aren't complete enough. I'm mobile, so can't test.
Hmm. It's a combination of both. There's no warning, but it's also so verbose that had there been a warning I could easily see having missed it:
make all-recursive make[1]: Entering directory '/home/torvalds/src/libdc' Making all in include make[2]: Entering directory '/home/torvalds/src/libdc/include' Making all in libdivecomputer make[3]: Entering directory '/home/torvalds/src/libdc/include/libdivecomputer' make[3]: Nothing to be done for 'all'. make[3]: Leaving directory '/home/torvalds/src/libdc/include/libdivecomputer' make[3]: Entering directory '/home/torvalds/src/libdc/include' make[3]: Nothing to be done for 'all-am'. make[3]: Leaving directory '/home/torvalds/src/libdc/include' make[2]: Leaving directory '/home/torvalds/src/libdc/include' Making all in src make[2]: Entering directory '/home/torvalds/src/libdc/src' make all-am make[3]: Entering directory '/home/torvalds/src/libdc/src' CC suunto_eonsteel_parser.lo CCLD libdivecomputer.la make[3]: Leaving directory '/home/torvalds/src/libdc/src' make[2]: Leaving directory '/home/torvalds/src/libdc/src' Making all in examples make[2]: Entering directory '/home/torvalds/src/libdc/examples' CCLD universal CCLD ostc-fwupdate make[2]: Leaving directory '/home/torvalds/src/libdc/examples' make[2]: Entering directory '/home/torvalds/src/libdc' make[2]: Leaving directory '/home/torvalds/src/libdc' make[1]: Leaving directory '/home/torvalds/src/libdc'
It's hard seeing the important stuff with all that noise. But no, there was no actual warning about unused functions either.
I don't do autoconf (tool of the devil, as far as I'm concerned), so I have no idea hoqw to get rid of the annoying directory entry crap, and the appatent lack of -Wall for he compiler. Oh well.
Linus