Re: [PATCH 3/4] tests: Move ARRAY_LENGTH to a shared header for reuse

2014-11-28 Thread Bryce Harrington
On Fri, Nov 28, 2014 at 12:41:21PM +0100, Marek Chalupa wrote: > IMO it'd be good to add guards (#ifndef ARRAY_LENGTH) around this macro, > because it is used extensively in wayland/weston and in the future it may > happen that > it'll be redefined by some included header. Right, I'll send a follo

Re: [PATCH 3/4] tests: Move ARRAY_LENGTH to a shared header for reuse

2014-11-28 Thread Marek Chalupa
IMO it'd be good to add guards (#ifndef ARRAY_LENGTH) around this macro, because it is used extensively in wayland/weston and in the future it may happen that it'll be redefined by some included header. Reviewed-by: Marek Chalupa On 20 November 2014 at 02:18, Bryce Harrington wrote: > Signed-o

[PATCH 3/4] tests: Move ARRAY_LENGTH to a shared header for reuse

2014-11-19 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- tests/presentation-test.c | 2 -- tests/weston-test-runner.h | 4 +++- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/presentation-test.c b/tests/presentation-test.c index b99e93d..f97e5b3 100644 --- a/tests/presentation-test.c +++ b/test