Re: [PATCH wayland 4/6] tests: Use volatile pointer for NULL dereference

2018-09-02 Thread Thiago Macieira
On Tuesday, 28 August 2018 23:17:13 PDT Daniel Stone wrote: > Clang warns that it can silently discard a non-volatile write to a NULL > pointer (perhaps it constitutes undefined behaviour?), and recommends It is. > changing it to volatile. That doesn't help. It's still UB and can be discarded

[PATCH wayland 4/6] tests: Use volatile pointer for NULL dereference

2018-08-29 Thread Daniel Stone
Clang warns that it can silently discard a non-volatile write to a NULL pointer (perhaps it constitutes undefined behaviour?), and recommends changing it to volatile. This patch slavishly complies with the demand of the unfeeling machine. Signed-off-by: Daniel Stone --- tests/sanity-test.c | 4