forked from Imagelibrary/rtems
libtest: Fix implicit type conversions
This fix relates to a Coverity issue (PW.MIXED_ENUM_TYPE).
This commit is contained in:
@@ -252,7 +252,7 @@ T_interrupt_test_change_state(T_interrupt_test_state expected_state,
|
|||||||
_Atomic_Compare_exchange_uint(&ctx->state, &expected,
|
_Atomic_Compare_exchange_uint(&ctx->state, &expected,
|
||||||
desired_state, ATOMIC_ORDER_RELAXED, ATOMIC_ORDER_RELAXED);
|
desired_state, ATOMIC_ORDER_RELAXED, ATOMIC_ORDER_RELAXED);
|
||||||
|
|
||||||
return expected;
|
return (T_interrupt_test_state)expected;
|
||||||
}
|
}
|
||||||
|
|
||||||
T_interrupt_test_state
|
T_interrupt_test_state
|
||||||
@@ -261,7 +261,8 @@ T_interrupt_test_get_state(void)
|
|||||||
T_interrupt_context *ctx;
|
T_interrupt_context *ctx;
|
||||||
|
|
||||||
ctx = &T_interrupt_instance;
|
ctx = &T_interrupt_instance;
|
||||||
return _Atomic_Load_uint(&ctx->state, ATOMIC_ORDER_RELAXED);
|
return (T_interrupt_test_state)_Atomic_Load_uint(&ctx->state,
|
||||||
|
ATOMIC_ORDER_RELAXED);
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
@@ -288,7 +289,8 @@ T_interrupt_thread_switch(Thread_Control *executing, Thread_Control *heir)
|
|||||||
if (ctx->self == executing) {
|
if (ctx->self == executing) {
|
||||||
T_interrupt_test_state state;
|
T_interrupt_test_state state;
|
||||||
|
|
||||||
state = _Atomic_Load_uint(&ctx->state, ATOMIC_ORDER_RELAXED);
|
state = (T_interrupt_test_state)_Atomic_Load_uint(&ctx->state,
|
||||||
|
ATOMIC_ORDER_RELAXED);
|
||||||
|
|
||||||
if (state != T_INTERRUPT_TEST_INITIAL) {
|
if (state != T_INTERRUPT_TEST_INITIAL) {
|
||||||
#ifdef RTEMS_SMP
|
#ifdef RTEMS_SMP
|
||||||
@@ -433,7 +435,8 @@ T_interrupt_test(const T_interrupt_test_config *config, void *arg)
|
|||||||
T_busy(busy);
|
T_busy(busy);
|
||||||
(*config->action)(arg);
|
(*config->action)(arg);
|
||||||
|
|
||||||
state = _Atomic_Exchange_uint(&ctx->state,
|
state = (T_interrupt_test_state)
|
||||||
|
_Atomic_Exchange_uint(&ctx->state,
|
||||||
T_INTERRUPT_TEST_INITIAL, ATOMIC_ORDER_RELAXED);
|
T_INTERRUPT_TEST_INITIAL, ATOMIC_ORDER_RELAXED);
|
||||||
|
|
||||||
if (state == T_INTERRUPT_TEST_DONE) {
|
if (state == T_INTERRUPT_TEST_DONE) {
|
||||||
|
|||||||
@@ -40,8 +40,8 @@ T_putchar_default(int c, void *arg)
|
|||||||
void
|
void
|
||||||
T_check_rsc(const T_check_context *t, uint32_t a, uint32_t e)
|
T_check_rsc(const T_check_context *t, uint32_t a, uint32_t e)
|
||||||
{
|
{
|
||||||
T_check(t, a == e, "%s == %s", rtems_status_text(a),
|
T_check(t, a == e, "%s == %s", rtems_status_text((rtems_status_code)a),
|
||||||
rtems_status_text(e));
|
rtems_status_text((rtems_status_code)e));
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
|
|||||||
Reference in New Issue
Block a user