Re: [PATCH wayland 3/4] tests: use test_set_timeout in display-test

2014-11-17 Thread Pekka Paalanen
On Wed, 12 Nov 2014 13:53:03 +0200 Giulio Camuffo wrote: > 2014-11-12 13:46 GMT+02:00 Marek Chalupa : > > > > > > On 7 November 2014 21:26, Giulio Camuffo > > wrote: > >> > >> I'm a bit confused here... you remove the alarm() calls but not the > >> signal handler or the sigaction setting. > > >

Re: [PATCH wayland 3/4] tests: use test_set_timeout in display-test

2014-11-12 Thread Giulio Camuffo
2014-11-12 13:46 GMT+02:00 Marek Chalupa : > > > On 7 November 2014 21:26, Giulio Camuffo wrote: >> >> I'm a bit confused here... you remove the alarm() calls but not the >> signal handler or the sigaction setting. > > > The alarm worked with default action - that is kill the process. There were >

Re: [PATCH wayland 3/4] tests: use test_set_timeout in display-test

2014-11-12 Thread Marek Chalupa
On 7 November 2014 21:26, Giulio Camuffo wrote: > I'm a bit confused here... you remove the alarm() calls but not the > signal handler or the sigaction setting. The alarm worked with default action - that is kill the process. There were no signal handler or sigaction setting before. When test f

Re: [PATCH wayland 3/4] tests: use test_set_timeout in display-test

2014-11-07 Thread Giulio Camuffo
I'm a bit confused here... you remove the alarm() calls but not the signal handler or the sigaction setting. On the other hand, in the patch introducing test_set_timeout() you add a new SIGALRM handler... Am i missing something? -- Giulio 2014-09-24 15:37 GMT+03:00 Marek Chalupa : > replace ca

[PATCH wayland 3/4] tests: use test_set_timeout in display-test

2014-09-24 Thread Marek Chalupa
replace call to alarm() with test_set_timeout() Signed-off-by: Marek Chalupa --- tests/display-test.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/display-test.c b/tests/display-test.c index a1e45b1..8435d60 100644 --- a/tests/display-test.c +++ b/tests/displ