Why not another style thread? (was Re: cvs commit: src/lib/libc/gen getgrent.c)

[Jacques A. Vidrine]

[This typo came from NetBSD, so in this particular source I have no
 intention of changing the style.]
 
What do folks think about

  1)    if (data)
                free(data);

versus

  2)    free(data);

versus

  3)    #define xfree(x) if ((x) != NULL) free(x);
        xfree(data);
-- 
Jacques Vidrine / [EMAIL PROTECTED] / [EMAIL PROTECTED] / [EMAIL PROTECTED]

On Sun, Dec 17, 2000 at 01:10:41PM -0800, Jacques Vidrine wrote:
> nectar      2000/12/17 13:10:41 PST
> 
>   Modified files:
>     lib/libc/gen         getgrent.c 
>   Log:
>   Fix mostly harmless typo:
>   
>       if (data);
>               free(data);
>   
>   Discovered by:      emacs cc-mode
>   
>   Revision  Changes    Path
>   1.19      +2 -2      src/lib/libc/gen/getgrent.c
> 
> 


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message