Hi Alexandros,

On 16 November 2017 at 16:20, Alexandros Frantzis
<alexandros.frant...@collabora.com> wrote:

> +ZUC_TEST(timespec_test, timespec_is_zero)
> +{
> +       struct timespec zero = { 0 };
> +       struct timespec non_zero_sec = { 1, 0 };
> +       struct timespec non_zero_nsec = { 0, 1 };
> +
The standard (be that POSIX or C99) does not guarantee the layout of
the struct. Hence the above approach may work, but it's a bad idea.
Personally I'm a fan of C99 designated initializers, although one
could set the tv_sec and tv_nsec individually.

Same comment applies through the rest of the series.

HTH
Emil
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to