[PATCH 17/19 v2] Set padding bytes to 0 in WriteToClient

Keith Packard keithp at keithp.com
Mon Jul 2 22:43:43 PDT 2012


Alan Coopersmith <alan.coopersmith at oracle.com> writes:

> Clear them out when needed instead of leaving whatever values were
> present in previously sent messages.
>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>
> v2: Simplified as suggested by Keith to just use memset instead of a 
> Duff-device style switch statement, since most calls won't need this.

Looks good.

Reviewed-by: Keith Packard <keithp at keithp.com>

Any response to my complaint about mixing initializers and struct value
assignment styles? I'm afraid I've scared you off...

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120702/4267f2d6/attachment.pgp>


More information about the xorg-devel mailing list