Dmitry Eremin-Solenikov(lumag) replied on github web page:
test/validation/api/crypto/odp_crypto_test_inp.c
@@ -270,6 +273,11 @@ static int alg_packet_op_enq(odp_packet_t pkt,
return rc;
}
+ if (!result.ok)
+ CU_ASSERT(odp_packet_has_error(pkt))
+ else
+ CU_ASSERT(!odp_packet_has_error(pkt));
+
Comment:
ok.
> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
> `CU_ASSERT(result.ok || odp_packet_has_error(pkt));` is simpler.
>> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
>> `CU_ASSERT()` needs to be enclosed in braces when used in an `if/else`
>> statement. However a simpler test is just:
>> ```
>> CU_ASSERT(result.ok != odp_packet_has_error(pkt));
>> ```
https://github.com/Linaro/odp/pull/294#discussion_r150434172
updated_at 2017-11-13 01:40:54
