[PATCH test 1/1] test/list: Fix test_xorg_list_del test
Jeremy Huddleston Sequoia
jeremyhu at apple.com
Tue Aug 28 12:46:52 PDT 2012
We never use child[3], so it's state is undefined.
This issue seems to have existed since the test was first
written: 92788e677be79bd04e5ef140f4ced50ad8b1bf8e
Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
CC: Peter Hutterer <peter.hutterer at who-t.net>
---
test/list.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/test/list.c b/test/list.c
index 82d2327..f9f54ee 100644
--- a/test/list.c
+++ b/test/list.c
@@ -137,7 +137,7 @@ static void
test_xorg_list_del(void)
{
struct parent parent = { 0 };
- struct child child[3];
+ struct child child[2];
struct child *c;
xorg_list_init(&parent.children);
@@ -178,8 +178,8 @@ test_xorg_list_del(void)
xorg_list_add(&child[0].node, &parent.children);
xorg_list_del(&parent.children);
assert(xorg_list_is_empty(&parent.children));
+ assert(!xorg_list_is_empty(&child[0].node));
assert(!xorg_list_is_empty(&child[1].node));
- assert(!xorg_list_is_empty(&child[2].node));
}
static void
--
1.7.11.5
More information about the xorg-devel
mailing list