All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tiwei Bie <tiwei.bie@intel.com>
To: Shreyansh Jain <shreyansh.jain@nxp.com>
Cc: dev@dpdk.org, thomas@monjalon.net, "Van Haaren,
	Harry" <harry.van.haaren@intel.com>
Subject: Re: [PATCH] all: refactor coding style
Date: Thu, 20 Jul 2017 13:53:10 +0800	[thread overview]
Message-ID: <20170720055309.GA975@debian-ZGViaWFuCg> (raw)
In-Reply-To: <39182ae5-d8e1-f84a-6701-2732dfbfdb5f@nxp.com>

On Thu, Jul 20, 2017 at 10:34:39AM +0530, Shreyansh Jain wrote:
> On Wednesday 19 July 2017 02:36 PM, Tiwei Bie wrote:
> > Remove the unwanted spaces before `;' across DPDK source code
> > by below one-liner with some minor manual refinements.
> > 
> > find . -name '*.[ch]' | xargs sed -i 's/\([^;(]\) \+;/\1;/g'
> > 
> > The fixes for cmdline library are skipped, because it has a
> > different coding style. It deserves a separate cleanup if
> > necessary. The fixes for drivers' base code are also skipped
> > to keep the base code intact.
> > 
> > Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
> > ---
[...]
> >   		/* compute the number of steps to the right */
> > diff --git a/lib/librte_sched/rte_bitmap.h b/lib/librte_sched/rte_bitmap.h
> > index 010d752..e487b58 100644
> > --- a/lib/librte_sched/rte_bitmap.h
> > +++ b/lib/librte_sched/rte_bitmap.h
> > @@ -500,7 +500,8 @@ __rte_bitmap_scan_read(struct rte_bitmap *bmp, uint32_t *pos, uint64_t *slab)
> >   	uint64_t *slab2;
> >   	slab2 = bmp->array2 + bmp->index2;
> > -	for ( ; bmp->go2 ; bmp->index2 ++, slab2 ++, bmp->go2 = bmp->index2 & RTE_BITMAP_CL_SLAB_MASK) {
> > +	for ( ; bmp->go2; bmp->index2++, slab2++,
> > +			  bmp->go2 = bmp->index2 & RTE_BITMAP_CL_SLAB_MASK) {
> 
>            ^^^^
> Trivial: space before ';' in 'for' here should also be removed.
> 

Thank you for your feedbacks! :-)

Hmm.. Actually the space between `(' and `;' was kept intentionally
when I wrote this 's/\([^;(]\) \+;/\1;/g' sed script. There are many
other such cases. It's acceptable to me, and I thought we like it:

diff --git i/app/test-eventdev/parser.h w/app/test-eventdev/parser.h
index 75a5a3b..372b85f 100644
--- i/app/test-eventdev/parser.h
+++ w/app/test-eventdev/parser.h
@@ -41,7 +41,7 @@
 #define skip_white_spaces(pos)			\
 ({						\
 	__typeof__(pos) _p = (pos);		\
-	for ( ; isspace(*_p); _p++)		\
+	for (; isspace(*_p); _p++)		\
 		;				\
 	_p;					\
 })
diff --git i/app/test-eventdev/test_perf_common.c w/app/test-eventdev/test_perf_common.c
index a5b768c..36b78bf 100644
--- i/app/test-eventdev/test_perf_common.c
+++ w/app/test-eventdev/test_perf_common.c
@@ -284,7 +284,7 @@ perf_event_dev_port_setup(struct evt_test *test, struct evt_options *opt,
 			.new_event_threshold = 1200,
 	};
 	prod = 0;
-	for ( ; port < perf_nb_event_ports(opt); port++) {
+	for (; port < perf_nb_event_ports(opt); port++) {
 		struct prod_data *p = &t->prod[port];
 
 		p->dev_id = opt->dev_id;
diff --git i/drivers/net/bonding/rte_eth_bond_pmd.c w/drivers/net/bonding/rte_eth_bond_pmd.c
index 383e27c..5d8e068 100644
--- i/drivers/net/bonding/rte_eth_bond_pmd.c
+++ w/drivers/net/bonding/rte_eth_bond_pmd.c
@@ -371,7 +371,7 @@ bond_ethdev_tx_burst_8023ad_fast_queue(void *queue, struct rte_mbuf **bufs,
 		/* If tx burst fails move packets to end of bufs */
 		if (unlikely(num_tx_slave < slave_nb_pkts[i])) {
 			uint16_t j = nb_pkts - num_tx_fail_total;
-			for ( ; num_tx_slave < slave_nb_pkts[i]; j++,
+			for (; num_tx_slave < slave_nb_pkts[i]; j++,
 					num_tx_slave++)
 				bufs[j] = slave_bufs[i][num_tx_slave];
 		}
@@ -1308,7 +1308,7 @@ bond_ethdev_tx_burst_8023ad(void *queue, struct rte_mbuf **bufs,
 				slave_bufs[i], slave_nb_pkts[i]);
 
 		/* If tx burst fails drop slow packets */
-		for ( ; num_tx_slave < slave_slow_nb_pkts[i]; num_tx_slave++)
+		for (; num_tx_slave < slave_slow_nb_pkts[i]; num_tx_slave++)
 			rte_pktmbuf_free(slave_bufs[i][num_tx_slave]);
 
 		num_tx_total += num_tx_slave - slave_slow_nb_pkts[i];
@@ -1317,7 +1317,7 @@ bond_ethdev_tx_burst_8023ad(void *queue, struct rte_mbuf **bufs,
 		/* If tx burst fails move packets to end of bufs */
 		if (unlikely(num_tx_slave < slave_nb_pkts[i])) {
 			uint16_t j = nb_pkts - num_tx_fail_total;
-			for ( ; num_tx_slave < slave_nb_pkts[i]; j++, num_tx_slave++)
+			for (; num_tx_slave < slave_nb_pkts[i]; j++, num_tx_slave++)
 				bufs[j] = slave_bufs[i][num_tx_slave];
 		}
 	}
diff --git i/drivers/net/cxgbe/base/t4_hw.c w/drivers/net/cxgbe/base/t4_hw.c
index a8ccea0..ed26f71 100644
--- i/drivers/net/cxgbe/base/t4_hw.c
+++ w/drivers/net/cxgbe/base/t4_hw.c
@@ -277,7 +277,7 @@ static void t4_report_fw_error(struct adapter *adap)
 static void get_mbox_rpl(struct adapter *adap, __be64 *rpl, int nflit,
 			 u32 mbox_addr)
 {
-	for ( ; nflit; nflit--, mbox_addr += 8)
+	for (; nflit; nflit--, mbox_addr += 8)
 		*rpl++ = htobe64(t4_read_reg64(adap, mbox_addr));
 }
 
@@ -2483,7 +2483,7 @@ int t4_read_flash(struct adapter *adapter, unsigned int addr,
 	if (ret != 0)
 		return ret;
 
-	for ( ; nwords; nwords--, data++) {
+	for (; nwords; nwords--, data++) {
 		ret = sf1_read(adapter, 4, nwords > 1, nwords == 1, data);
 		if (nwords == 1)
 			t4_write_reg(adapter, A_SF_OP, 0);    /* unlock SF */
diff --git i/drivers/net/e1000/igb_flow.c w/drivers/net/e1000/igb_flow.c
index db73b18..f9b32ca 100644
--- i/drivers/net/e1000/igb_flow.c
+++ w/drivers/net/e1000/igb_flow.c
@@ -1160,7 +1160,7 @@ cons_parse_flex_filter(const struct rte_flow_attr *attr,
 
 	i = 0;
 
-	for ( ; shift < CHAR_BIT; shift++) {
+	for (; shift < CHAR_BIT; shift++) {
 		filter->mask[j] |= (0x80 >> shift);
 		i++;
 		if (i == raw_spec->length)
diff --git i/examples/ip_pipeline/cpu_core_map.c w/examples/ip_pipeline/cpu_core_map.c
index 38c5786..45752f7 100644
--- i/examples/ip_pipeline/cpu_core_map.c
+++ w/examples/ip_pipeline/cpu_core_map.c
@@ -235,7 +235,7 @@ cpu_core_map_compute_and_check(struct cpu_core_map *map)
 					return -1;
 			}
 
-		for ( ; core_id < map->n_max_cores_per_socket; core_id++)
+		for (; core_id < map->n_max_cores_per_socket; core_id++)
 			for (ht_id = 0;
 				ht_id < map->n_max_ht_per_core;
 				ht_id++) {
diff --git i/examples/ip_pipeline/parser.c w/examples/ip_pipeline/parser.c
index 689e206..c57d161 100644
--- i/examples/ip_pipeline/parser.c
+++ w/examples/ip_pipeline/parser.c
@@ -380,7 +380,7 @@ parse_mpls_labels(char *string, uint32_t *labels, uint32_t *n_labels)
 	}
 
 	/* At least one label should be present */
-	for ( ; (*string != '\0'); ) {
+	for (; (*string != '\0'); ) {
 		char *next;
 		int value;
 
diff --git i/examples/ip_pipeline/parser.h w/examples/ip_pipeline/parser.h
index 9bd36af..2aa232e 100644
--- i/examples/ip_pipeline/parser.h
+++ w/examples/ip_pipeline/parser.h
@@ -44,7 +44,7 @@
 #define skip_white_spaces(pos)			\
 ({						\
 	__typeof__(pos) _p = (pos);		\
-	for ( ; isspace(*_p); _p++)		\
+	for (; isspace(*_p); _p++)		\
 		;				\
 	_p;					\
 })
diff --git i/examples/ip_pipeline/pipeline/pipeline_actions_common.h w/examples/ip_pipeline/pipeline/pipeline_actions_common.h
index ab08612..aa4c682 100644
--- i/examples/ip_pipeline/pipeline/pipeline_actions_common.h
+++ w/examples/ip_pipeline/pipeline/pipeline_actions_common.h
@@ -53,7 +53,7 @@ f_ah(									\
 	for (i = 0; i < (n_pkts & (~0x3LLU)); i += 4)			\
 		f_pkt4_work(&pkts[i], arg);				\
 									\
-	for ( ; i < n_pkts; i++)					\
+	for (; i < n_pkts; i++)					\
 		f_pkt_work(pkts[i], arg);				\
 									\
 	return 0;							\
@@ -75,7 +75,7 @@ f_ah(									\
 	for (i = 0; i < (n_pkts & (~0x3LLU)); i += 4)	\
 		f_pkt4_work(&pkts[i], arg);				\
 									\
-	for ( ; i < n_pkts; i++)				\
+	for (; i < n_pkts; i++)				\
 		f_pkt_work(pkts[i], arg);			\
 									\
 	return 0;							\
@@ -97,10 +97,10 @@ f_ah(									\
 		for (i = 0; i < (n_pkts & (~0x3LLU)); i += 4)		\
 			f_pkt4_work(&pkts[i], &entries[i], arg);	\
 									\
-		for ( ; i < n_pkts; i++)				\
+		for (; i < n_pkts; i++)				\
 			f_pkt_work(pkts[i], entries[i], arg);		\
 	} else								\
-		for ( ; pkts_in_mask; ) {				\
+		for (; pkts_in_mask; ) {				\
 			uint32_t pos = __builtin_ctzll(pkts_in_mask);	\
 			uint64_t pkt_mask = 1LLU << pos;		\
 									\
@@ -127,10 +127,10 @@ f_ah(									\
 		for (i = 0; i < (n_pkts & (~0x3LLU)); i += 4)		\
 			f_pkt4_work(&pkts[i], entry, arg);		\
 									\
-		for ( ; i < n_pkts; i++)				\
+		for (; i < n_pkts; i++)				\
 			f_pkt_work(pkts[i], entry, arg);		\
 	} else								\
-		for ( ; pkts_in_mask; ) {				\
+		for (; pkts_in_mask; ) {				\
 			uint32_t pos = __builtin_ctzll(pkts_in_mask);	\
 			uint64_t pkt_mask = 1LLU << pos;		\
 									\
@@ -164,13 +164,13 @@ f_ah(									\
 			pkts_out_mask ^= mask << i;			\
 		}							\
 									\
-		for ( ; i < n_pkts; i++) {				\
+		for (; i < n_pkts; i++) {				\
 			uint64_t mask = f_pkt_work(pkts[i],		\
 				entries[i], arg, time);			\
 			pkts_out_mask ^= mask << i;			\
 		}							\
 	} else								\
-		for ( ; pkts_in_mask; ) {				\
+		for (; pkts_in_mask; ) {				\
 			uint32_t pos = __builtin_ctzll(pkts_in_mask);	\
 			uint64_t pkt_mask = 1LLU << pos;		\
 			uint64_t mask = f_pkt_work(pkts[pos],		\
@@ -208,12 +208,12 @@ f_ah(									\
 			pkts_out_mask ^= mask << i;			\
 		}							\
 									\
-		for ( ; i < n_pkts; i++) {				\
+		for (; i < n_pkts; i++) {				\
 			uint64_t mask = f_pkt_work(pkts[i], entry, arg, time);\
 			pkts_out_mask ^= mask << i;			\
 		}							\
 	} else								\
-		for ( ; pkts_in_mask; ) {				\
+		for (; pkts_in_mask; ) {				\
 			uint32_t pos = __builtin_ctzll(pkts_in_mask);	\
 			uint64_t pkt_mask = 1LLU << pos;		\
 			uint64_t mask = f_pkt_work(pkts[pos],		\
diff --git i/examples/ip_pipeline/pipeline/pipeline_common_be.c w/examples/ip_pipeline/pipeline/pipeline_common_be.c
index 347e72b..4831713 100644
--- i/examples/ip_pipeline/pipeline/pipeline_common_be.c
+++ w/examples/ip_pipeline/pipeline/pipeline_common_be.c
@@ -180,7 +180,7 @@ pipeline_msg_req_handle(struct pipeline *p)
 	uint32_t msgq_id;
 
 	for (msgq_id = 0; msgq_id < p->n_msgq; msgq_id++) {
-		for ( ; ; ) {
+		for (; ; ) {
 			struct pipeline_msg_req *req;
 			pipeline_msg_req_handler f_handle;
 
diff --git i/examples/ip_pipeline/pipeline/pipeline_common_fe.c w/examples/ip_pipeline/pipeline/pipeline_common_fe.c
index 7521187..d800911 100644
--- i/examples/ip_pipeline/pipeline/pipeline_common_fe.c
+++ w/examples/ip_pipeline/pipeline/pipeline_common_fe.c
@@ -61,7 +61,7 @@ app_pipeline_track_pktq_out_to_link(struct app_params *app,
 	if (p == NULL)
 		return NULL;
 
-	for ( ; ; ) {
+	for (; ; ) {
 		struct app_pktq_out_params *pktq_out =
 			&p->pktq_out[pktq_out_id];
 
diff --git i/examples/load_balancer/runtime.c w/examples/load_balancer/runtime.c
index e54b785..3983b17 100644
--- i/examples/load_balancer/runtime.c
+++ w/examples/load_balancer/runtime.c
@@ -272,7 +272,7 @@ app_lcore_io_rx(
 		}
 
 		/* Handle the last 1, 2 (when n_mbufs is even) or 3 (when n_mbufs is odd) packets  */
-		for ( ; j < n_mbufs; j += 1) {
+		for (; j < n_mbufs; j += 1) {
 			struct rte_mbuf *mbuf;
 			uint8_t *data;
 			uint32_t worker;
@@ -465,7 +465,7 @@ app_lcore_main_loop_io(void)
 
 	uint8_t pos_lb = app.pos_lb;
 
-	for ( ; ; ) {
+	for (; ; ) {
 		if (APP_LCORE_IO_FLUSH && (unlikely(i == APP_LCORE_IO_FLUSH))) {
 			if (likely(lp->rx.n_nic_queues > 0)) {
 				app_lcore_io_rx_flush(lp, n_workers);
@@ -636,7 +636,7 @@ app_lcore_main_loop_worker(void) {
 	uint32_t bsz_rd = app.burst_size_worker_read;
 	uint32_t bsz_wr = app.burst_size_worker_write;
 
-	for ( ; ; ) {
+	for (; ; ) {
 		if (APP_LCORE_WORKER_FLUSH && (unlikely(i == APP_LCORE_WORKER_FLUSH))) {
 			app_lcore_worker_flush(lp);
 			i = 0;
diff --git i/lib/librte_distributor/rte_distributor.c w/lib/librte_distributor/rte_distributor.c
index 4c09e25..9992e00 100644
--- i/lib/librte_distributor/rte_distributor.c
+++ w/lib/librte_distributor/rte_distributor.c
@@ -347,7 +347,7 @@ release(struct rte_distributor *d, unsigned int wkr)
 		d->in_flight_tags[wkr][i] = d->backlog[wkr].tags[i];
 	}
 	buf->count = i;
-	for ( ; i < RTE_DIST_BURST_SIZE; i++) {
+	for (; i < RTE_DIST_BURST_SIZE; i++) {
 		buf->bufptr64[i] = RTE_DISTRIB_GET_BUF;
 		d->in_flight_tags[wkr][i] = 0;
 	}
diff --git i/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h w/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h
index c3a2619..e93c915 100644
--- i/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h
+++ w/lib/librte_eal/common/include/arch/arm/rte_memcpy_32.h
@@ -215,7 +215,7 @@ rte_memcpy_func(void *dst, const void *src, size_t n)
 	 * copies was found to be faster than doing 128 and 32 byte copies as
 	 * well.
 	 */
-	for ( ; n >= 256; n -= 256) {
+	for (; n >= 256; n -= 256) {
 		rte_mov256((uint8_t *)dst, (const uint8_t *)src);
 		dst = (uint8_t *)dst + 256;
 		src = (const uint8_t *)src + 256;
diff --git i/lib/librte_eal/common/include/arch/ppc_64/rte_memcpy.h w/lib/librte_eal/common/include/arch/ppc_64/rte_memcpy.h
index ca9d1dc..e763589 100644
--- i/lib/librte_eal/common/include/arch/ppc_64/rte_memcpy.h
+++ w/lib/librte_eal/common/include/arch/ppc_64/rte_memcpy.h
@@ -154,7 +154,7 @@ rte_memcpy_func(void *dst, const void *src, size_t n)
 	 * copies was found to be faster than doing 128 and 32 byte copies as
 	 * well.
 	 */
-	for ( ; n >= 256; n -= 256) {
+	for (; n >= 256; n -= 256) {
 		rte_mov256((uint8_t *)dst, (const uint8_t *)src);
 		dst = (uint8_t *)dst + 256;
 		src = (const uint8_t *)src + 256;
diff --git i/lib/librte_ether/rte_ethdev.c w/lib/librte_ether/rte_ethdev.c
index 937f30b..fbd28e5 100644
--- i/lib/librte_ether/rte_ethdev.c
+++ w/lib/librte_ether/rte_ethdev.c
@@ -1817,7 +1817,7 @@ rte_eth_xstats_get(uint8_t port_id, struct rte_eth_xstat *xstats,
 	for (i = 0; i < count; i++)
 		xstats[i].id = i;
 	/* add an offset to driver-specific stats */
-	for ( ; i < count + xcount; i++)
+	for (; i < count + xcount; i++)
 		xstats[i].id += count;
 
 	return count + xcount;
diff --git i/lib/librte_port/rte_port_ethdev.c w/lib/librte_port/rte_port_ethdev.c
index d5c5fba..8fdb75f 100644
--- i/lib/librte_port/rte_port_ethdev.c
+++ w/lib/librte_port/rte_port_ethdev.c
@@ -202,7 +202,7 @@ send_burst(struct rte_port_ethdev_writer *p)
 			 p->tx_buf, p->tx_buf_count);
 
 	RTE_PORT_ETHDEV_WRITER_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -246,13 +246,13 @@ rte_port_ethdev_writer_tx_bulk(void *port,
 			n_pkts);
 
 		RTE_PORT_ETHDEV_WRITER_STATS_PKTS_DROP_ADD(p, n_pkts - n_pkts_ok);
-		for ( ; n_pkts_ok < n_pkts; n_pkts_ok++) {
+		for (; n_pkts_ok < n_pkts; n_pkts_ok++) {
 			struct rte_mbuf *pkt = pkts[n_pkts_ok];
 
 			rte_pktmbuf_free(pkt);
 		}
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
@@ -408,7 +408,7 @@ send_burst_nodrop(struct rte_port_ethdev_writer_nodrop *p)
 
 	/* We didn't send the packets in maximum allowed attempts */
 	RTE_PORT_ETHDEV_WRITER_NODROP_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -465,7 +465,7 @@ rte_port_ethdev_writer_nodrop_tx_bulk(void *port,
 		}
 		send_burst_nodrop(p);
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_fd.c w/lib/librte_port/rte_port_fd.c
index b5b3729..58364cf 100644
--- i/lib/librte_port/rte_port_fd.c
+++ w/lib/librte_port/rte_port_fd.c
@@ -275,7 +275,7 @@ rte_port_fd_writer_tx_bulk(void *port,
 			p->tx_buf[tx_buf_count++] = pkts[i];
 		RTE_PORT_FD_WRITER_STATS_PKTS_IN_ADD(p, n_pkts);
 	} else
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
@@ -412,7 +412,7 @@ send_burst_nodrop(struct rte_port_fd_writer_nodrop *p)
 		void *pkt_data = rte_pktmbuf_mtod(pkt, void*);
 		size_t n_bytes = rte_pktmbuf_data_len(pkt);
 
-		for ( ; n_retries < p->n_retries; n_retries++) {
+		for (; n_retries < p->n_retries; n_retries++) {
 			ssize_t ret;
 
 			ret = write(p->fd, pkt_data, n_bytes);
@@ -460,7 +460,7 @@ rte_port_fd_writer_nodrop_tx_bulk(void *port,
 			p->tx_buf[tx_buf_count++] = pkts[i];
 		RTE_PORT_FD_WRITER_NODROP_STATS_PKTS_IN_ADD(p, n_pkts);
 	} else
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_frag.c w/lib/librte_port/rte_port_frag.c
index a00c9ae..a0c1339 100644
--- i/lib/librte_port/rte_port_frag.c
+++ w/lib/librte_port/rte_port_frag.c
@@ -178,7 +178,7 @@ rte_port_ring_reader_frag_rx(void *port,
 	p->n_frags = 0;
 
 	/* Look to "pkts" buffer to get more packets */
-	for ( ; ; ) {
+	for (; ; ) {
 		struct rte_mbuf *pkt;
 		uint32_t n_pkts_to_provide, i;
 		int status;
diff --git i/lib/librte_port/rte_port_kni.c w/lib/librte_port/rte_port_kni.c
index 2515fb2..1aac316 100644
--- i/lib/librte_port/rte_port_kni.c
+++ w/lib/librte_port/rte_port_kni.c
@@ -400,7 +400,7 @@ send_burst_nodrop(struct rte_port_kni_writer_nodrop *p)
 
 	/* We didn't send the packets in maximum allowed attempts */
 	RTE_PORT_KNI_WRITER_NODROP_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -456,7 +456,7 @@ rte_port_kni_writer_nodrop_tx_bulk(void *port,
 		}
 		send_burst_nodrop(p);
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_ras.c w/lib/librte_port/rte_port_ras.c
index 415fadd..beee7d7 100644
--- i/lib/librte_port/rte_port_ras.c
+++ w/lib/librte_port/rte_port_ras.c
@@ -170,7 +170,7 @@ send_burst(struct rte_port_ring_writer_ras *p)
 			p->tx_buf_count, NULL);
 
 	RTE_PORT_RING_WRITER_RAS_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -274,7 +274,7 @@ rte_port_ring_writer_ras_tx_bulk(void *port,
 				send_burst(p);
 		}
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_ring.c w/lib/librte_port/rte_port_ring.c
index a4e709c..2c62dea 100644
--- i/lib/librte_port/rte_port_ring.c
+++ w/lib/librte_port/rte_port_ring.c
@@ -246,7 +246,7 @@ send_burst(struct rte_port_ring_writer *p)
 			p->tx_buf_count, NULL);
 
 	RTE_PORT_RING_WRITER_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -261,7 +261,7 @@ send_burst_mp(struct rte_port_ring_writer *p)
 			p->tx_buf_count, NULL);
 
 	RTE_PORT_RING_WRITER_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -327,13 +327,13 @@ rte_port_ring_writer_tx_bulk_internal(void *port,
 					(void **)pkts, n_pkts, NULL);
 
 		RTE_PORT_RING_WRITER_STATS_PKTS_DROP_ADD(p, n_pkts - n_pkts_ok);
-		for ( ; n_pkts_ok < n_pkts; n_pkts_ok++) {
+		for (; n_pkts_ok < n_pkts; n_pkts_ok++) {
 			struct rte_mbuf *pkt = pkts[n_pkts_ok];
 
 			rte_pktmbuf_free(pkt);
 		}
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
@@ -541,7 +541,7 @@ send_burst_nodrop(struct rte_port_ring_writer_nodrop *p)
 
 	/* We didn't send the packets in maximum allowed attempts */
 	RTE_PORT_RING_WRITER_NODROP_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -575,7 +575,7 @@ send_burst_mp_nodrop(struct rte_port_ring_writer_nodrop *p)
 
 	/* We didn't send the packets in maximum allowed attempts */
 	RTE_PORT_RING_WRITER_NODROP_STATS_PKTS_DROP_ADD(p, p->tx_buf_count - nb_tx);
-	for ( ; nb_tx < p->tx_buf_count; nb_tx++)
+	for (; nb_tx < p->tx_buf_count; nb_tx++)
 		rte_pktmbuf_free(p->tx_buf[nb_tx]);
 
 	p->tx_buf_count = 0;
@@ -661,7 +661,7 @@ rte_port_ring_writer_nodrop_tx_bulk_internal(void *port,
 		else
 			send_burst_nodrop(p);
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_sched.c w/lib/librte_port/rte_port_sched.c
index 9100a19..5b0d3e1 100644
--- i/lib/librte_port/rte_port_sched.c
+++ w/lib/librte_port/rte_port_sched.c
@@ -233,7 +233,7 @@ rte_port_sched_writer_tx_bulk(void *port,
 		nb_tx = rte_sched_port_enqueue(p->sched, pkts, n_pkts);
 		RTE_PORT_SCHED_WRITER_STATS_PKTS_DROP_ADD(p, n_pkts - nb_tx);
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_port/rte_port_source_sink.c w/lib/librte_port/rte_port_source_sink.c
index a79f2f6..947174f 100644
--- i/lib/librte_port/rte_port_source_sink.c
+++ w/lib/librte_port/rte_port_source_sink.c
@@ -559,7 +559,7 @@ rte_port_sink_tx_bulk(void *port, struct rte_mbuf **pkts,
 			uint64_t dump_pkts_mask = pkts_mask;
 			uint32_t pkt_index;
 
-			for ( ; dump_pkts_mask; ) {
+			for (; dump_pkts_mask; ) {
 				pkt_index = __builtin_ctzll(
 					dump_pkts_mask);
 				PCAP_SINK_WRITE_PKT(p, pkts[pkt_index]);
@@ -567,7 +567,7 @@ rte_port_sink_tx_bulk(void *port, struct rte_mbuf **pkts,
 			}
 		}
 
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_sched/rte_bitmap.h w/lib/librte_sched/rte_bitmap.h
index e487b58..c1a0ac3 100644
--- i/lib/librte_sched/rte_bitmap.h
+++ w/lib/librte_sched/rte_bitmap.h
@@ -500,7 +500,7 @@ __rte_bitmap_scan_read(struct rte_bitmap *bmp, uint32_t *pos, uint64_t *slab)
 	uint64_t *slab2;
 
 	slab2 = bmp->array2 + bmp->index2;
-	for ( ; bmp->go2; bmp->index2++, slab2++,
+	for (; bmp->go2; bmp->index2++, slab2++,
 			  bmp->go2 = bmp->index2 & RTE_BITMAP_CL_SLAB_MASK) {
 		if (*slab2) {
 			*pos = bmp->index2 << RTE_BITMAP_SLAB_BIT_SIZE_LOG2;
diff --git i/lib/librte_table/rte_table_array.c w/lib/librte_table/rte_table_array.c
index cf7be88..0f61da2 100644
--- i/lib/librte_table/rte_table_array.c
+++ w/lib/librte_table/rte_table_array.c
@@ -194,7 +194,7 @@ rte_table_array_lookup(
 				t->entry_size];
 		}
 	} else {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			uint32_t pkt_index = __builtin_ctzll(pkts_mask);
 			uint64_t pkt_mask = 1LLU << pkt_index;
 			struct rte_mbuf *pkt = pkts[pkt_index];
diff --git i/lib/librte_table/rte_table_hash_ext.c w/lib/librte_table/rte_table_hash_ext.c
index e718102..bc6c105 100644
--- i/lib/librte_table/rte_table_hash_ext.c
+++ w/lib/librte_table/rte_table_hash_ext.c
@@ -445,7 +445,7 @@ static int rte_table_hash_ext_lookup_unoptimized(
 
 	__rte_unused uint32_t n_pkts_in = __builtin_popcountll(pkts_mask);
 
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		struct bucket *bkt0, *bkt;
 		struct rte_mbuf *pkt;
 		uint8_t *key;
@@ -912,7 +912,7 @@ static int rte_table_hash_ext_lookup(
 	* Pipeline run
 	*
 	*/
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		pkt30_index = pkt20_index;
 		pkt31_index = pkt21_index;
@@ -1049,7 +1049,7 @@ static int rte_table_hash_ext_lookup_dosig(
 	* Pipeline run
 	*
 	*/
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		pkt30_index = pkt20_index;
 		pkt31_index = pkt21_index;
diff --git i/lib/librte_table/rte_table_hash_key16.c w/lib/librte_table/rte_table_hash_key16.c
index ce057b7..3bc5f76 100644
--- i/lib/librte_table/rte_table_hash_key16.c
+++ w/lib/librte_table/rte_table_hash_key16.c
@@ -920,7 +920,7 @@ rte_table_hash_lookup_key16_lru(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_16 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -962,7 +962,7 @@ rte_table_hash_lookup_key16_lru(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1049,7 +1049,7 @@ rte_table_hash_lookup_key16_lru_dosig(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_16 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1091,7 +1091,7 @@ rte_table_hash_lookup_key16_lru_dosig(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1179,7 +1179,7 @@ rte_table_hash_lookup_key16_ext(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_16 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1219,7 +1219,7 @@ rte_table_hash_lookup_key16_ext(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1284,10 +1284,10 @@ rte_table_hash_lookup_key16_ext(
 
 grind_next_buckets:
 	/* Grind next buckets */
-	for ( ; buckets_mask; ) {
+	for (; buckets_mask; ) {
 		uint64_t buckets_mask_next = 0;
 
-		for ( ; buckets_mask; ) {
+		for (; buckets_mask; ) {
 			uint64_t pkt_mask;
 			uint32_t pkt_index;
 
@@ -1331,7 +1331,7 @@ rte_table_hash_lookup_key16_ext_dosig(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_16 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1371,7 +1371,7 @@ rte_table_hash_lookup_key16_ext_dosig(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1436,10 +1436,10 @@ rte_table_hash_lookup_key16_ext_dosig(
 
 grind_next_buckets:
 	/* Grind next buckets */
-	for ( ; buckets_mask; ) {
+	for (; buckets_mask; ) {
 		uint64_t buckets_mask_next = 0;
 
-		for ( ; buckets_mask; ) {
+		for (; buckets_mask; ) {
 			uint64_t pkt_mask;
 			uint32_t pkt_index;
 
diff --git i/lib/librte_table/rte_table_hash_key32.c w/lib/librte_table/rte_table_hash_key32.c
index 31fe6fd..4b3330a 100644
--- i/lib/librte_table/rte_table_hash_key32.c
+++ w/lib/librte_table/rte_table_hash_key32.c
@@ -850,7 +850,7 @@ rte_table_hash_lookup_key32_lru(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_32 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -891,7 +891,7 @@ rte_table_hash_lookup_key32_lru(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -979,7 +979,7 @@ rte_table_hash_lookup_key32_ext(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_32 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1019,7 +1019,7 @@ rte_table_hash_lookup_key32_ext(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1084,10 +1084,10 @@ rte_table_hash_lookup_key32_ext(
 
 grind_next_buckets:
 	/* Grind next buckets */
-	for ( ; buckets_mask; ) {
+	for (; buckets_mask; ) {
 		uint64_t buckets_mask_next = 0;
 
-		for ( ; buckets_mask; ) {
+		for (; buckets_mask; ) {
 			uint64_t pkt_mask;
 			uint32_t pkt_index;
 
diff --git i/lib/librte_table/rte_table_hash_key8.c w/lib/librte_table/rte_table_hash_key8.c
index 5f0c656..5a61e3d 100644
--- i/lib/librte_table/rte_table_hash_key8.c
+++ w/lib/librte_table/rte_table_hash_key8.c
@@ -880,7 +880,7 @@ rte_table_hash_lookup_key8_lru(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_8 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -921,7 +921,7 @@ rte_table_hash_lookup_key8_lru(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1006,7 +1006,7 @@ rte_table_hash_lookup_key8_lru_dosig(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_8 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1047,7 +1047,7 @@ rte_table_hash_lookup_key8_lru_dosig(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1134,7 +1134,7 @@ rte_table_hash_lookup_key8_ext(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_8 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1174,7 +1174,7 @@ rte_table_hash_lookup_key8_ext(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1239,10 +1239,10 @@ rte_table_hash_lookup_key8_ext(
 
 grind_next_buckets:
 	/* Grind next buckets */
-	for ( ; buckets_mask; ) {
+	for (; buckets_mask; ) {
 		uint64_t buckets_mask_next = 0;
 
-		for ( ; buckets_mask; ) {
+		for (; buckets_mask; ) {
 			uint64_t pkt_mask;
 			uint32_t pkt_index;
 
@@ -1284,7 +1284,7 @@ rte_table_hash_lookup_key8_ext_dosig(
 
 	/* Cannot run the pipeline with less than 5 packets */
 	if (__builtin_popcountll(pkts_mask) < 5) {
-		for ( ; pkts_mask; ) {
+		for (; pkts_mask; ) {
 			struct rte_bucket_4_8 *bucket;
 			struct rte_mbuf *mbuf;
 			uint32_t pkt_index;
@@ -1324,7 +1324,7 @@ rte_table_hash_lookup_key8_ext_dosig(
 	 * Pipeline run
 	 *
 	 */
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		bucket20 = bucket10;
 		bucket21 = bucket11;
@@ -1389,10 +1389,10 @@ rte_table_hash_lookup_key8_ext_dosig(
 
 grind_next_buckets:
 	/* Grind next buckets */
-	for ( ; buckets_mask; ) {
+	for (; buckets_mask; ) {
 		uint64_t buckets_mask_next = 0;
 
-		for ( ; buckets_mask; ) {
+		for (; buckets_mask; ) {
 			uint64_t pkt_mask;
 			uint32_t pkt_index;
 
diff --git i/lib/librte_table/rte_table_hash_lru.c w/lib/librte_table/rte_table_hash_lru.c
index 5a4864e..b24fceb 100644
--- i/lib/librte_table/rte_table_hash_lru.c
+++ w/lib/librte_table/rte_table_hash_lru.c
@@ -374,7 +374,7 @@ static int rte_table_hash_lru_lookup_unoptimized(
 	__rte_unused uint32_t n_pkts_in = __builtin_popcountll(pkts_mask);
 	RTE_TABLE_HASH_LRU_STATS_PKTS_IN_ADD(t, n_pkts_in);
 
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		struct bucket *bkt;
 		struct rte_mbuf *pkt;
 		uint8_t *key;
@@ -853,7 +853,7 @@ static int rte_table_hash_lru_lookup(
 	* Pipeline run
 	*
 	*/
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		pkt30_index = pkt20_index;
 		pkt31_index = pkt21_index;
@@ -986,7 +986,7 @@ static int rte_table_hash_lru_lookup_dosig(
 	* Pipeline run
 	*
 	*/
-	for ( ; pkts_mask; ) {
+	for (; pkts_mask; ) {
 		/* Pipeline feed */
 		pkt30_index = pkt20_index;
 		pkt31_index = pkt21_index;
diff --git i/lib/librte_timer/rte_timer.c w/lib/librte_timer/rte_timer.c
index 5ee0840..c21b51d 100644
--- i/lib/librte_timer/rte_timer.c
+++ w/lib/librte_timer/rte_timer.c
@@ -562,7 +562,7 @@ void rte_timer_manage(void)
 	run_first_tim = tim;
 	pprev = &run_first_tim;
 
-	for ( ; tim != NULL; tim = next_tim) {
+	for (; tim != NULL; tim = next_tim) {
 		next_tim = tim->sl_next[0];
 
 		ret = timer_set_running_state(tim);
diff --git i/test/test-pipeline/pipeline_acl.c w/test/test-pipeline/pipeline_acl.c
index 22d5f36..dfd476f 100644
--- i/test/test-pipeline/pipeline_acl.c
+++ w/test/test-pipeline/pipeline_acl.c
@@ -264,7 +264,7 @@ app_main_loop_worker_pipeline_acl(void) {
 
 	/* Run-time */
 #if APP_FLUSH == 0
-	for ( ; ; )
+	for (; ; )
 		rte_pipeline_run(p);
 #else
 	for (i = 0; ; i++) {
diff --git i/test/test-pipeline/pipeline_hash.c w/test/test-pipeline/pipeline_hash.c
index 991e381..0bcaa69 100644
--- i/test/test-pipeline/pipeline_hash.c
+++ w/test/test-pipeline/pipeline_hash.c
@@ -463,7 +463,7 @@ app_main_loop_worker_pipeline_hash(void) {
 
 	/* Run-time */
 #if APP_FLUSH == 0
-	for ( ; ; )
+	for (; ; )
 		rte_pipeline_run(p);
 #else
 	for (i = 0; ; i++) {
diff --git i/test/test-pipeline/pipeline_lpm.c w/test/test-pipeline/pipeline_lpm.c
index ecea6b3..051b43b 100644
--- i/test/test-pipeline/pipeline_lpm.c
+++ w/test/test-pipeline/pipeline_lpm.c
@@ -189,7 +189,7 @@ app_main_loop_worker_pipeline_lpm(void) {
 
 	/* Run-time */
 #if APP_FLUSH == 0
-	for ( ; ; )
+	for (; ; )
 		rte_pipeline_run(p);
 #else
 	for (i = 0; ; i++) {
diff --git i/test/test-pipeline/pipeline_lpm_ipv6.c w/test/test-pipeline/pipeline_lpm_ipv6.c
index 3352e89..9b9709c 100644
--- i/test/test-pipeline/pipeline_lpm_ipv6.c
+++ w/test/test-pipeline/pipeline_lpm_ipv6.c
@@ -187,7 +187,7 @@ app_main_loop_worker_pipeline_lpm_ipv6(void) {
 
 	/* Run-time */
 #if APP_FLUSH == 0
-	for ( ; ; )
+	for (; ; )
 		rte_pipeline_run(p);
 #else
 	for (i = 0; ; i++) {
diff --git i/test/test-pipeline/pipeline_stub.c w/test/test-pipeline/pipeline_stub.c
index ba710ca..0f899b8 100644
--- i/test/test-pipeline/pipeline_stub.c
+++ w/test/test-pipeline/pipeline_stub.c
@@ -151,7 +151,7 @@ app_main_loop_worker_pipeline_stub(void) {
 
 	/* Run-time */
 #if APP_FLUSH == 0
-	for ( ; ; )
+	for (; ; )
 		rte_pipeline_run(p);
 #else
 	for (i = 0; ; i++) {

Best regards,
Tiwei Bie

  reply	other threads:[~2017-07-20  5:52 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-19  9:06 [PATCH] all: refactor coding style Tiwei Bie
2017-07-19  9:24 ` Van Haaren, Harry
2017-07-19 10:23   ` Tiwei Bie
2017-07-20  7:56     ` Jens Freimann
2017-07-20  8:32       ` Thomas Monjalon
2017-07-20  9:01         ` Tiwei Bie
2017-07-19 10:45 ` Trahe, Fiona
2017-07-20  5:04 ` Shreyansh Jain
2017-07-20  5:53   ` Tiwei Bie [this message]
2017-07-20  7:13     ` Shreyansh Jain

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170720055309.GA975@debian-ZGViaWFuCg \
    --to=tiwei.bie@intel.com \
    --cc=dev@dpdk.org \
    --cc=harry.van.haaren@intel.com \
    --cc=shreyansh.jain@nxp.com \
    --cc=thomas@monjalon.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.