On Wed, 2007-05-02 at 09:49 +0200, Bernd Petrovitsch wrote:
> On Tue, 2007-05-01 at 15:08 -0700, Patrick McCarty wrote:
[...]
> > Second, in src/socket_parser.c Line 197ish was changed recently (rev1.24) from:
> >  static struct tms tms_buf;
> >  [...]
> >  now_times = times(&tms_buf);
> > to
> >  now_times = times(NULL);
> 
> Probably because no one used tms_buf (and glibc and/or the Linux kernel
> is usually graceful on passed NULL pointers). However, reading the man
> page once more, I find:
> ----  snip  ----
>        On Linux, the buf argument can be specified as NULL,  with  the  result  that  times()  just
>        returns  a  function result.  However, POSIX does not specify this behaviour, and most other
>        Unix implementations require a non-NULL value for buf.
> ----  snip  ----
> I better readd it anyways with a comment .....

Done.
It was the only instance with "NULL" as parameter and I added the
comment also to the other calls (which are only in scheduler.h).

	Bernd
-- 
Firmix Software GmbH                   http://www.firmix.at/
mobil: +43 664 4416156                 fax: +43 1 7890849-55
          Embedded Linux Development and Services



