> -----Original Message-----
> From: Jastrzebski, MichalX K
> Sent: Friday, September 29, 2017 8:57 PM
> To: Wu, Jingjing <[email protected]>
> Cc: [email protected]; Jain, Deepak K <[email protected]>; Kulasek, TomaszX
> <[email protected]>; Chilikin, Andrey <[email protected]>;
> [email protected]
> Subject: [PATCH] app/testpmd: fix runtime dead code
> 
> From: Tomasz Kulasek <[email protected]>
> 
> Coverity reports DEADCODE, as assumes that RTE_LIBRTE_I40E_PMD
> is defined and function compiles entirely.
> The fix is about to place printf function into #else branch
> of conditional compilation, since it is known at compile
> time if i40e is supported. No need to check it at runtime.
> 
> Coverity issue: 158646
> Fixes: 37a56cce7b8e ("app/testpmd: enable DDP get info feature")
> Cc: [email protected]
> Cc: [email protected]
> 
> Signed-off-by: Tomasz Kulasek <[email protected]>
Acked-by: Jingjing Wu <[email protected]>

Reply via email to