I agree with Bill. retval should be tested for success in this case. Regards, Bala
On 14 August 2015 at 01:24, Bill Fischofer <bill.fischo...@linaro.org> wrote: > > > On Thu, Aug 13, 2015 at 2:04 PM, Mike Holmes <mike.hol...@linaro.org> > wrote: > >> retval is not used, remove it >> >> Signed-off-by: Mike Holmes <mike.hol...@linaro.org> >> --- >> test/validation/classification/odp_classification_tests.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/test/validation/classification/odp_classification_tests.c >> b/test/validation/classification/odp_classification_tests.c >> index 0e0c4eb..827afa4 100644 >> --- a/test/validation/classification/odp_classification_tests.c >> +++ b/test/validation/classification/odp_classification_tests.c >> @@ -405,7 +405,7 @@ void configure_cls_pmr_chain(void) >> &qparam); >> CU_ASSERT_FATAL(queue_list[CLS_PMR_CHAIN_DST] != >> ODP_QUEUE_INVALID); >> >> - retval = odp_cos_set_queue(cos_list[CLS_PMR_CHAIN_DST], >> + odp_cos_set_queue(cos_list[CLS_PMR_CHAIN_DST], >> queue_list[CLS_PMR_CHAIN_DST]); >> > > Again the issue here is not that retval is unused but that what's missing > is the following: > > CU_ASSERT(retval == 0); > >> >> parse_ipv4_string(CLS_PMR_CHAIN_SADDR, &addr, &mask); >> -- >> 2.1.4 >> >> _______________________________________________ >> lng-odp mailing list >> lng-odp@lists.linaro.org >> https://lists.linaro.org/mailman/listinfo/lng-odp >> > >
_______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp