[dpdk-dev] [PATCH v5 1/3] app/testpmd: move dumping packets to a separate function

[Iremonger, Bernard]

Hi Raslan,

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Wednesday, October 17, 2018 1:28 PM
> To: Raslan Darawsheh <rasland at mellanox.com>; Wu, Jingjing
> <jingjing.wu at intel.com>
> Cc: thomas at monjalon.net; dev at dpdk.org; shahafs at mellanox.com;
> xuemingl at mellanox.com; orika at mellanox.com;
> jerin.jacob at caviumnetworks.com; david.marchand at 6wind.com; Iremonger,
> Bernard <bernard.iremonger at intel.com>
> Subject: Re: [PATCH v5 1/3] app/testpmd: move dumping packets to a separate
> function
> 
> On 10/17/2018 8:27 AM, Raslan Darawsheh wrote:
> > verbosity for the received/sent packets is needed in all of the
> > forwarding engines so moving it to be in a separate function
> >
> > Acked-by: Bernard Iremonger <bernard.iremonger at intel.com>
> > Signed-off-by: Raslan Darawsheh <rasland at mellanox.com>
> 
> <...>
> 
> > +static  inline void
> > +dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[],
> > +	      uint16_t nb_pkts, int is_rx)
> > +{
> 
> This is causing build error when only this patch applied:
> 
> .../app/test-pmd/util.c:25:1: error: unused function 'dump_pkt_burst'
> [-Werror,-Wunused-function]
> dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf *pkts[], ^

Builds ok with gcc but fails with clang

Regards,

Bernard.