Re: [PATCH 5/7] drivers/net: fix unused but set variables

[Stephen Hemminger]

On Fri, 12 Nov 2021 16:38:37 +0000
"Walsh, Conor" <conor.wa...@intel.com> wrote:

> Would something along the lines of this make more sense?
> 
> #ifdef RTE_LIBRTE_VMXNET3_DEBUG_TX
> completed += vmxnet3_unmap_pkt(tcd->txdIdx, txq);
> #else
> vmxnet3_unmap_pkt(tcd->txdIdx, txq);
> #endif
> 
> Thanks,
> Conor.

Why not just
        RTE_USED(completed);

#ifdef's introduce more testing issues.