From b2b77956dc3e2123b8aa25a1195014bea7e0ee10 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Mon, 20 Jul 2009 14:51:21 +0000 Subject: [PATCH] 2009-07-20 Joel Sherrill * score/src/corebarrierwait.c: Reverse order of tests to increase test coverage. --- cpukit/ChangeLog | 5 +++++ cpukit/score/src/corebarrierwait.c | 6 +++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/cpukit/ChangeLog b/cpukit/ChangeLog index f6d79950f0..039f90527e 100644 --- a/cpukit/ChangeLog +++ b/cpukit/ChangeLog @@ -1,3 +1,8 @@ +2009-07-20 Joel Sherrill + + * score/src/corebarrierwait.c: Reverse order of tests to increase test + coverage. + 2009-07-19 Joel Sherrill * rtems/src/ratemongetstatus.c: Fix spacing. diff --git a/cpukit/score/src/corebarrierwait.c b/cpukit/score/src/corebarrierwait.c index 07e8fcdc88..9e71a8ac9a 100644 --- a/cpukit/score/src/corebarrierwait.c +++ b/cpukit/score/src/corebarrierwait.c @@ -59,9 +59,9 @@ void _CORE_barrier_Wait( executing->Wait.return_code = CORE_BARRIER_STATUS_SUCCESSFUL; _ISR_Disable( level ); the_barrier->number_of_waiting_threads++; - if ( the_barrier->number_of_waiting_threads == - the_barrier->Attributes.maximum_count) { - if ( _CORE_barrier_Is_automatic( &the_barrier->Attributes ) ) { + if ( _CORE_barrier_Is_automatic( &the_barrier->Attributes ) ) { + if ( the_barrier->number_of_waiting_threads == + the_barrier->Attributes.maximum_count) { executing->Wait.return_code = CORE_BARRIER_STATUS_AUTOMATICALLY_RELEASED; _ISR_Enable( level ); _CORE_barrier_Release( the_barrier, id, api_barrier_mp_support );