the restrict thing is probably good/fine to do.
   
   Also, the struct timeval is making use of the long type while posix specifies 
   a couple of other data types for the timeval structure. Should this be 
   changed?
   
   i.e.
   struct timeval {
   -       long    tv_sec;         /* seconds */
   -       long    tv_usec;        /* and microseconds */
   +       time_t          tv_sec;         /* seconds */
   +       suseconds_t     tv_usec;        /* and microseconds */
    };
   
   Please cc me in any replies as I am no longer on any of the NetBSD mailing 
   lists.


this however is wrong.  time_t and long, and i'd guess suseconds_t and
long both are difference sizes.  ie, this breaks the ABI and API.


.mrg.