Re: Remove useless NULL casts from mv(1)

[Philip Guenther]

On Mon, Sep 14, 2015 at 3:39 PM, Michael McConville
<mmcco...@sccs.swarthmore.edu> wrote:
...
> -               execl(_PATH_CP, "cp", "-PRp", "--", from, to, (char *)NULL);
> +               execl(_PATH_CP, "cp", "-PRp", "--", from, to, NULL);

Nope, this is a case in C where the cast is necessary.  To quote style(9):

     NULL is the preferred null pointer constant.  Use NULL instead of
     (type *)0 or (type *)NULL in all cases except for arguments to variadic
     functions where the compiler does not know the type.


Philip Guenther