Hello. On Thu, 2014-01-23 at 19:01, Cedric BAIL wrote: > cedric pushed a commit to branch master. > > http://git.enlightenment.org/core/efl.git/commit/?id=f5c125dfc286ffc596e2acfe3fde65be41f0e9d9 > > commit f5c125dfc286ffc596e2acfe3fde65be41f0e9d9 > Author: Cedric BAIL <[email protected]> > Date: Fri Jan 24 12:00:32 2014 +0900 > > eina: fix test suite for rwlock... hopefully.
While most people would ignore thsi commit I have to say I find it very exciting to have the spurious jenkins fails gone for good ... hopefully. :) regards Stefan Schmidt > src/tests/eina/eina_test_lock.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/src/tests/eina/eina_test_lock.c b/src/tests/eina/eina_test_lock.c > index 7c6626e..1b50f6e 100644 > --- a/src/tests/eina/eina_test_lock.c > +++ b/src/tests/eina/eina_test_lock.c > @@ -152,7 +152,11 @@ _eina_test_rwlock_thread(void *data EINA_UNUSED, > Eina_Thread t EINA_UNUSED) > counter = 7200; > fail_if(eina_rwlock_release(&mutex) != EINA_LOCK_SUCCEED); > > + fail_if(!eina_barrier_wait(&barrier)); > + > + fail_if(eina_lock_take(&mtcond) != EINA_LOCK_SUCCEED); > fail_if(!eina_condition_broadcast(&cond)); > + fail_if(eina_lock_release(&mtcond) != EINA_LOCK_SUCCEED); > > return NULL; > } > @@ -183,14 +187,16 @@ START_TEST(eina_test_rwlock) > fail_if(eina_rwlock_release(&mutex) != EINA_LOCK_SUCCEED); > > fail_if(eina_lock_take(&mtcond) != EINA_LOCK_SUCCEED); > + fail_if(!eina_barrier_wait(&barrier)); > + > fail_if(!eina_condition_wait(&cond)); > fail_if(eina_lock_release(&mtcond) != EINA_LOCK_SUCCEED); > - > + > fail_if(eina_rwlock_take_read(&mutex) != EINA_LOCK_SUCCEED); > fail_if(counter != 7200); > fail_if(eina_rwlock_release(&mutex) != EINA_LOCK_SUCCEED); > > - eina_condition_timedwait(&cond, 0.01); > + /* eina_condition_timedwait(&cond, 0.01); */ > eina_thread_join(thread); > > eina_condition_free(&cond); > > -- > > ------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
