From 34345a9b69bbf5ccd74a3a75cf3cf102a735546a Mon Sep 17 00:00:00 2001 From: Pablo de Lara Date: Tue, 17 Apr 2018 14:13:42 +0100 Subject: [PATCH] eventdev: fix build with icc ICC complains about variable being used before its value is set. Since the variable is only assigned in the for loop, its declaration is moved inside and is initialized. lib/librte_eventdev/rte_event_timer_adapter.c(708): error #592: variable "ret" is used before its value is set RTE_SET_USED(ret); Fixes: 6750b21bd6af ("eventdev: add default software timer adapter") Signed-off-by: Pablo de Lara Acked-by: Ferruh Yigit Acked-by: Erik Gabriel Carrillo --- lib/librte_eventdev/rte_event_timer_adapter.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/librte_eventdev/rte_event_timer_adapter.c b/lib/librte_eventdev/rte_event_timer_adapter.c index 6eba6b44d7..9a863f5cb6 100644 --- a/lib/librte_eventdev/rte_event_timer_adapter.c +++ b/lib/librte_eventdev/rte_event_timer_adapter.c @@ -695,7 +695,7 @@ check_destination_event_queue(struct rte_event_timer *evtim, static int sw_event_timer_adapter_service_func(void *arg) { - int ret, i, num_msgs; + int i, num_msgs; uint64_t cycles, opaque; uint16_t nb_evs_flushed = 0; uint16_t nb_evs_invalid = 0; @@ -705,8 +705,6 @@ sw_event_timer_adapter_service_func(void *arg) struct rte_timer *tim = NULL; struct msg *msg, *msgs[NB_OBJS]; - RTE_SET_USED(ret); - adapter = arg; sw_data = adapter->data->adapter_priv; @@ -720,6 +718,10 @@ sw_event_timer_adapter_service_func(void *arg) (void **)msgs, NB_OBJS, NULL); for (i = 0; i < num_msgs; i++) { + int ret = 0; + + RTE_SET_USED(ret); + msg = msgs[i]; evtim = msg->evtim; -- 2.20.1