[PATCH] tevent: Do not use uninitialized tv in py_tevent_context_add_timer
Douglas Bagnall
douglas.bagnall at catalyst.net.nz
Mon Jan 28 02:09:12 UTC 2019
hi Lukas,
On 21/01/19 11:12 PM, Lukas Slebodnik wrote:
> In another word.
> "{ 0 };" is a quick shorthand for setting the entire structure to zero.
> And it is used in recent commits in samba
>
> So I am not clear what is wrong from C99 POV here.
>
You are right of course.
I was perhaps distracted by the `gcc -Wmissing-field-initializers`
pseudo-standard, but in any case that is happy with "{ 0 };" and so it
should be.
So RB+, though it would still be nice if you could initialise callback
to NULL.
Another reviewer?
Douglas
More information about the samba-technical
mailing list