Code Review / vpp.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 5d0d549) | patch
string_test: Coverity woe 59/17759/1
authorSteven Luong <sluong@cisco.com>
Thu, 21 Feb 2019 19:58:14 +0000 (11:58 -0800)
committerSteven Luong <sluong@cisco.com>
Thu, 21 Feb 2019 20:01:10 +0000 (12:01 -0800)
commit3be3cd60bc41d095174ea413aa753a7bd9eff73e
treea89ab62f78c743934b143ec0fec4d99a688123a0tree | snapshot
parent5d0d5494db58422eb528c0f8b39a86ea966505e9commit | diff
string_test: Coverity woe

Coverity complains about dead code as shown below and it is right.
The fix is to simply remove the dead code.

503  if (v_indicator != indicator)

CID 190173 (#3 of 3): Logically dead code (DEADCODE)
dead_error_line: Execution cannot reach this statement: return -1;.
504    return -1;

Change-Id: Ibca9e10451a4459db099bef5ecc6939474bdb903
Signed-off-by: Steven Luong <sluong@cisco.com>
src/plugins/unittest/string_test.c diff | blob | history
Vector Packet Processing