On Mon, Jun 15, 2015 at 09:56:18PM +0200, Fritjof Bornebusch wrote:

Ping ....

> Hi tech@,
> 
> mark this unlink(2) call as *(void)*, as there is no need to check the return 
> value.
> This makes it more consistent to all other unlink(2) calls, since they are 
> marked as *(void)* as
> well.
> 
> Regards,
> --F.
> 
> 
> Index: co.c
> ===================================================================
> RCS file: /cvs/src/usr.bin/rcs/co.c,v
> retrieving revision 1.121
> diff -u -p -r1.121 co.c
> --- co.c      13 Jun 2015 20:15:21 -0000      1.121
> +++ co.c      15 Jun 2015 19:50:12 -0000
> @@ -553,7 +553,7 @@ checkout_file_has_diffs(RCSFILE *rfp, RC
>       ret = diffreg(dst, tempfile, bp, D_FORCEASCII);
>  
>       buf_free(bp);
> -     unlink(tempfile);
> +     (void)unlink(tempfile);
>       free(tempfile);
>  
>       return (ret);
> 

Reply via email to