[Xcb] [PATCH] Initialize all fields of addrinfo and use #ifdef

Julien Danjou julien at danjou.info
Thu Aug 28 04:47:13 PDT 2008


At 1219854880 time_t, Barton C Massey wrote:
> The memset() here seems like it could potentially mask a
> bug.  Initializing unknown extra fields to zero seems
> scary---better to leave them uninitialized and hope the
> compiler or valgrind can notice and warn. Am I missing
> something?

I agree, but there _is_ a warning from the compiler.
So if initializing to 0 is wrong, what are the good values?

I let you think about this. ;-)
Meanwhile, I postpone this patch.

Cheers,
-- 
Julien Danjou
// ᐰ <julien at danjou.info>   http://julien.danjou.info
// 9A0D 5FD9 EB42 22F6 8974  C95C A462 B51E C2FE E5CD
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
Url : http://lists.freedesktop.org/archives/xcb/attachments/20080828/36f87528/attachment.pgp 


More information about the Xcb mailing list