[tz] [PATCH 1/3] Assume C89.

Paul_Koning at Dell.com Paul_Koning at Dell.com
Fri Oct 12 17:09:15 UTC 2012


On Oct 12, 2012, at 12:24 PM, Todd C. Miller wrote:

> For this bit in date.c:
> 
> -       found = select(FD_SETSIZE, &ready, (fd_set *)0, (fd_set *)0, &tout);
> +       found = select(FD_SETSIZE, &ready, 0, 0, &tout);
> 
> You probably want to use NULL, not a bare 0 for the 3rd and 4th
> arguments.

Makes no difference; NULL is just a macro that translates to 0, sometimes with a mostly-unnecessary cast to void*.

	paul





More information about the tz mailing list