[PATCH 3/4] queue-test: Add another assertion

Dima Ryazanov dima at gmail.com
Sun Sep 28 20:48:39 PDT 2014


I've brought this up once, but looks like it's acceptable in the test suite
since it already relies on asserts:
http://lists.freedesktop.org/archives/wayland-devel/2013-February/007454.html
On Sep 28, 2014 6:57 PM, "Bill Spitzak" <spitzak at gmail.com> wrote:

> On 09/28/2014 11:49 AM, Karsten Otto wrote:
>
>  -       wl_display_roundtrip(display);
>> +       assert(wl_display_roundtrip(display) != -1);
>>
>
> You can't put code that you require to run in an assert.
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140928/ae7236ce/attachment.html>


More information about the wayland-devel mailing list