[PATCH:libXTrap 2/2] Remove unnecessary memset return value casts to void.

Rémi Cardona remi at gentoo.org
Sat Jul 21 01:31:29 PDT 2012


Le 20/07/2012 23:41, Thomas Klausner a écrit :
> @@ -336,7 +336,7 @@ int XETrapSetMaxPacket(XETC *tc, Bool set_flag, CARD16 size)
>      XETCValues tcv;
>      int status = True;
>      
> -    (void)memset((char *)&tcv,0L,sizeof(tcv));
> +    memset((char *)&tcv,0L,sizeof(tcv));
              ^^^^^^^^

That cast is useless as well since the first argument of memset() is a
void*... But then again there are probably a hundred other things to
clean up in there.

Reviewed-by: Rémi Cardona <remi.cardona at free.fr>

Cheers


More information about the xorg-devel mailing list