Have you looked at the differences between using gcc -Wall and g++
(without -Wall)? It really does let a lot of things slip by. From
init/main.c,
g++ caught several additional types of (small?) errors: not handling
values falling through switch statements, comparisons between signed and
unsigned ints, unused parameters galore, and "illegal" conversion from
int (*)(void*) to int(*)(char*). While none of these are really
critical and C probably handles them all for us, it wouldn't hurt to
make the types more clear and attempt to handle illegal inputs to
procedures. Makes reading the code and tracking down bugs easier.