Hi,
yes, this is normal in tests - the fact is that without assert the tests
won't compile anyway, as Otto wrote, so the asserts are always present
there.
This patch is:
Reviewed-by: Marek Chalupa
On 29 September 2014 08:09, Karsten Otto wrote:
> Indeed, I just followed the lead of the other
Indeed, I just followed the lead of the other tests, which use assert exactly
like this. I think what makes it OK are these lines in test-runner.h:
#ifdef NDEBUG
#error "Tests must not be built with NDEBUG defined, they rely on assert()."
#endif
Cheers, Karsten
Am 29.09.2014 um 05:48 schrieb Di
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" wrote:
> On 09/28/2014 11:49 AM, Karsten Otto wrote:
>
> - w
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@lists.freed
From: Philip Withnall
Ensure that the round trip succeeds.
[KAO: adjusted to current test framework]
Signed-off-by: Philip Withnall
---
tests/queue-test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/queue-test.c b/tests/queue-test.c
index 96f2100..6e2e932 100644
