"Julio M. Merino Vidal" wrote:

> >From a very quick review, I see:
> - Auxiliary functions are not static.

Why do they need to be static?  While some of our source tree does this,
it's not the most common practice.

> - Functions that return values which are discarded should be prefixed
>   with (void).  I.e., (void)printf("blah");

I've always thought that "(void)printf(..." was overkill.  In our source
tree, a quick grep shows 71520 printf's without void and 2203 with void.

Simon.
--
Simon Burge                            <simonb@wasabisystems.com>
NetBSD Support and Service:         http://www.wasabisystems.com/