Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/27051/?format=api
{ "id": 27051, "url": "http://patches.dpdk.org/api/patches/27051/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1500455196-182365-1-git-send-email-tiwei.bie@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1500455196-182365-1-git-send-email-tiwei.bie@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1500455196-182365-1-git-send-email-tiwei.bie@intel.com", "date": "2017-07-19T09:06:36", "name": "[dpdk-dev] all: refactor coding style", "commit_ref": null, "pull_url": null, "state": "rejected", "archived": true, "hash": "96953ed0e85d50866683d3d490a6c110339999d8", "submitter": { "id": 617, "url": "http://patches.dpdk.org/api/people/617/?format=api", "name": "Tiwei Bie", "email": "tiwei.bie@intel.com" }, "delegate": null, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1500455196-182365-1-git-send-email-tiwei.bie@intel.com/mbox/", "series": [], "comments": "http://patches.dpdk.org/api/patches/27051/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/27051/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [IPv6:::1])\n\tby dpdk.org (Postfix) with ESMTP id 2E5183977;\n\tWed, 19 Jul 2017 11:11:04 +0200 (CEST)", "from mga01.intel.com (mga01.intel.com [192.55.52.88])\n\tby dpdk.org (Postfix) with ESMTP id C16C9374F\n\tfor <dev@dpdk.org>; Wed, 19 Jul 2017 11:11:01 +0200 (CEST)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n\tby fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t19 Jul 2017 02:11:00 -0700", "from dpdk25.sh.intel.com ([10.67.111.96])\n\tby fmsmga005.fm.intel.com with ESMTP; 19 Jul 2017 02:10:59 -0700" ], "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.40,380,1496127600\"; d=\"scan'208\";a=\"128459657\"", "From": "Tiwei Bie <tiwei.bie@intel.com>", "To": "dev@dpdk.org", "Cc": "thomas@monjalon.net", "Date": "Wed, 19 Jul 2017 17:06:36 +0800", "Message-Id": "<1500455196-182365-1-git-send-email-tiwei.bie@intel.com>", "X-Mailer": "git-send-email 2.7.4", "Subject": "[dpdk-dev] [PATCH] all: refactor coding style", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<http://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://dpdk.org/ml/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<http://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Remove the unwanted spaces before `;' across DPDK source code\nby below one-liner with some minor manual refinements.\n\nfind . -name '*.[ch]' | xargs sed -i 's/\\([^;(]\\) \\+;/\\1;/g'\n\nThe fixes for cmdline library are skipped, because it has a\ndifferent coding style. It deserves a separate cleanup if\nnecessary. The fixes for drivers' base code are also skipped\nto keep the base code intact.\n\nSigned-off-by: Tiwei Bie <tiwei.bie@intel.com>\n---\n app/test-pmd/testpmd.h | 4 ++--\n drivers/crypto/qat/qat_adf/icp_qat_fw.h | 2 +-\n drivers/event/dpaa2/dpaa2_eventdev.c | 2 +-\n drivers/mempool/dpaa2/dpaa2_hw_mempool.c | 2 +-\n drivers/net/bnx2x/bnx2x.c | 3 ++-\n drivers/net/bnx2x/elink.h | 2 +-\n drivers/net/e1000/igb_pf.c | 2 +-\n drivers/net/ena/ena_ethdev.c | 4 ++--\n drivers/net/qede/qede_ethdev.c | 2 +-\n drivers/net/vhost/rte_eth_vhost.c | 2 +-\n drivers/net/virtio/virtio_rxtx.c | 4 ++--\n drivers/net/xenvirt/rte_eth_xenvirt.c | 4 ++--\n drivers/net/xenvirt/rte_xen_lib.c | 2 +-\n drivers/net/xenvirt/virtqueue.h | 2 +-\n examples/ip_pipeline/cpu_core_map.c | 4 ++--\n examples/multi_process/l2fwd_fork/main.c | 2 +-\n examples/netmap_compat/lib/compat_netmap.c | 2 +-\n examples/performance-thread/l3fwd-thread/main.c | 2 +-\n examples/qos_sched/app_thread.c | 2 +-\n examples/quota_watermark/qw/main.c | 2 +-\n examples/vhost_xen/xenstore_parse.c | 3 +--\n lib/librte_distributor/rte_distributor.c | 12 +++++-----\n lib/librte_eal/linuxapp/eal/eal_memory.c | 2 +-\n lib/librte_eal/linuxapp/eal/eal_xen_memory.c | 2 +-\n lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c | 4 ++--\n .../linuxapp/kni/ethtool/ixgbe/ixgbe_main.c | 2 +-\n .../linuxapp/kni/ethtool/ixgbe/kcompat.c | 2 +-\n lib/librte_ether/rte_ethdev.c | 2 +-\n lib/librte_sched/rte_approx.c | 8 +++----\n lib/librte_sched/rte_bitmap.h | 3 ++-\n test/test/test_cryptodev.c | 2 +-\n test/test/test_cryptodev_perf.c | 26 +++++++++++-----------\n test/test/test_eventdev_sw.c | 2 +-\n test/test/test_malloc.c | 4 ++--\n test/test/test_memory.c | 2 +-\n test/test/test_mempool.c | 2 +-\n test/test/test_ring.c | 6 ++---\n test/test/test_table_acl.c | 2 +-\n test/test/test_table_pipeline.c | 2 +-\n 39 files changed, 69 insertions(+), 70 deletions(-)", "diff": "diff --git a/app/test-pmd/testpmd.h b/app/test-pmd/testpmd.h\nindex c9d7739..8f88d70 100644\n--- a/app/test-pmd/testpmd.h\n+++ b/app/test-pmd/testpmd.h\n@@ -118,8 +118,8 @@ struct fwd_stream {\n \tunsigned int rx_packets; /**< received packets */\n \tunsigned int tx_packets; /**< received packets transmitted */\n \tunsigned int fwd_dropped; /**< received packets not forwarded */\n-\tunsigned int rx_bad_ip_csum ; /**< received packets has bad ip checksum */\n-\tunsigned int rx_bad_l4_csum ; /**< received packets has bad l4 checksum */\n+\tunsigned int rx_bad_ip_csum; /**< received packets has bad ip checksum */\n+\tunsigned int rx_bad_l4_csum; /**< received packets has bad l4 checksum */\n #ifdef RTE_TEST_PMD_RECORD_CORE_CYCLES\n \tuint64_t core_cycles; /**< used for RX and TX processing */\n #endif\ndiff --git a/drivers/crypto/qat/qat_adf/icp_qat_fw.h b/drivers/crypto/qat/qat_adf/icp_qat_fw.h\nindex 5de34d5..c80989b 100644\n--- a/drivers/crypto/qat/qat_adf/icp_qat_fw.h\n+++ b/drivers/crypto/qat/qat_adf/icp_qat_fw.h\n@@ -51,7 +51,7 @@\n \n #define QAT_FIELD_SET(flags, val, bitpos, mask) \\\n { (flags) = (((flags) & (~((mask) << (bitpos)))) | \\\n-\t\t(((val) & (mask)) << (bitpos))) ; }\n+\t\t(((val) & (mask)) << (bitpos))); }\n \n #define QAT_FIELD_GET(flags, bitpos, mask) \\\n \t(((flags) >> (bitpos)) & (mask))\ndiff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c\nindex ed57376..89bc1ce 100644\n--- a/drivers/event/dpaa2/dpaa2_eventdev.c\n+++ b/drivers/event/dpaa2/dpaa2_eventdev.c\n@@ -587,7 +587,7 @@ dpaa2_eventdev_setup_dpci(struct dpaa2_dpci_dev *dpci_dev,\n \tdpci_dev->queue[DPAA2_EVENT_DPCI_ATOMIC_QUEUE].cb =\n \t\tdpaa2_eventdev_process_atomic;\n \n-\tfor (i = 0 ; i < DPAA2_EVENT_DPCI_MAX_QUEUES; i++) {\n+\tfor (i = 0; i < DPAA2_EVENT_DPCI_MAX_QUEUES; i++) {\n \t\trx_queue_cfg.user_ctx = (uint64_t)(&dpci_dev->queue[i]);\n \t\tret = dpci_set_rx_queue(&dpci_dev->dpci,\n \t\t\t\t\tCMD_PRI_LOW,\ndiff --git a/drivers/mempool/dpaa2/dpaa2_hw_mempool.c b/drivers/mempool/dpaa2/dpaa2_hw_mempool.c\nindex 3b2e5a9..31e0fe0 100644\n--- a/drivers/mempool/dpaa2/dpaa2_hw_mempool.c\n+++ b/drivers/mempool/dpaa2/dpaa2_hw_mempool.c\n@@ -204,7 +204,7 @@ rte_dpaa2_mbuf_release(struct rte_mempool *pool __rte_unused,\n \t\tgoto aligned;\n \n \t/* convert mbuf to buffers for the remainder */\n-\tfor (i = 0; i < n ; i++) {\n+\tfor (i = 0; i < n; i++) {\n #ifdef RTE_LIBRTE_DPAA2_USE_PHYS_IOVA\n \t\tbufs[i] = (uint64_t)rte_mempool_virt2phy(pool, obj_table[i])\n \t\t\t\t+ meta_data_size;\ndiff --git a/drivers/net/bnx2x/bnx2x.c b/drivers/net/bnx2x/bnx2x.c\nindex 06733d1..51989aa 100644\n--- a/drivers/net/bnx2x/bnx2x.c\n+++ b/drivers/net/bnx2x/bnx2x.c\n@@ -11535,7 +11535,8 @@ static int cut_gzip_prefix(const uint8_t * zbuf, int len)\n \t}\n \t/* file name is present */\n \tif (zbuf[3] & 0x8) {\n-\t\twhile ((zbuf[n++] != 0) && (n < len)) ;\n+\t\twhile ((zbuf[n++] != 0) && (n < len))\n+\t\t\t;\n \t}\n \n \treturn n;\ndiff --git a/drivers/net/bnx2x/elink.h b/drivers/net/bnx2x/elink.h\nindex 9401b7c..8f34177 100644\n--- a/drivers/net/bnx2x/elink.h\n+++ b/drivers/net/bnx2x/elink.h\n@@ -552,7 +552,7 @@ enum elink_cos_state {\n };\n \n struct elink_ets_cos_params {\n-\tenum elink_cos_state state ;\n+\tenum elink_cos_state state;\n \tunion {\n \t\tstruct elink_ets_bw_params bw_params;\n \t\tstruct elink_ets_sp_params sp_params;\ndiff --git a/drivers/net/e1000/igb_pf.c b/drivers/net/e1000/igb_pf.c\nindex 6809d30..52dab47 100644\n--- a/drivers/net/e1000/igb_pf.c\n+++ b/drivers/net/e1000/igb_pf.c\n@@ -203,7 +203,7 @@ int igb_pf_host_configure(struct rte_eth_dev *eth_dev)\n \t * enable vlan filtering and allow all vlan tags through\n \t */\n \tvlanctrl = E1000_READ_REG(hw, E1000_RCTL);\n-\tvlanctrl |= E1000_RCTL_VFE ; /* enable vlan filters */\n+\tvlanctrl |= E1000_RCTL_VFE; /* enable vlan filters */\n \tE1000_WRITE_REG(hw, E1000_RCTL, vlanctrl);\n \n \t/* VFTA - enable all vlan filters */\ndiff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c\nindex 80ce1f3..fde9c7e 100644\n--- a/drivers/net/ena/ena_ethdev.c\n+++ b/drivers/net/ena/ena_ethdev.c\n@@ -466,7 +466,7 @@ static int ena_rss_reta_update(struct rte_eth_dev *dev,\n \t\tgoto err;\n \t}\n \n-\tfor (i = 0 ; i < reta_size ; i++) {\n+\tfor (i = 0; i < reta_size; i++) {\n \t\t/* each reta_conf is for 64 entries.\n \t\t * to support 128 we use 2 conf of 64\n \t\t */\n@@ -525,7 +525,7 @@ static int ena_rss_reta_query(struct rte_eth_dev *dev,\n \t\tgoto err;\n \t}\n \n-\tfor (i = 0 ; i < reta_size ; i++) {\n+\tfor (i = 0; i < reta_size; i++) {\n \t\treta_conf_idx = i / RTE_RETA_GROUP_SIZE;\n \t\treta_idx = i % RTE_RETA_GROUP_SIZE;\n \t\tif (TEST_BIT(reta_conf[reta_conf_idx].mask, reta_idx))\ndiff --git a/drivers/net/qede/qede_ethdev.c b/drivers/net/qede/qede_ethdev.c\nindex a0616a4..e6f772d 100644\n--- a/drivers/net/qede/qede_ethdev.c\n+++ b/drivers/net/qede/qede_ethdev.c\n@@ -1774,7 +1774,7 @@ int qede_rss_hash_update(struct rte_eth_dev *eth_dev,\n \trss_params.rss_table_size_log = 7;\n \tvport_update_params.vport_id = 0;\n \t/* pass the L2 handles instead of qids */\n-\tfor (i = 0 ; i < ECORE_RSS_IND_TABLE_SIZE ; i++) {\n+\tfor (i = 0; i < ECORE_RSS_IND_TABLE_SIZE; i++) {\n \t\tidx = qdev->rss_ind_table[i];\n \t\trss_params.rss_ind_table[i] = qdev->fp_array[idx].rxq->handle;\n \t}\ndiff --git a/drivers/net/vhost/rte_eth_vhost.c b/drivers/net/vhost/rte_eth_vhost.c\nindex 0dac5e6..a8c7173 100644\n--- a/drivers/net/vhost/rte_eth_vhost.c\n+++ b/drivers/net/vhost/rte_eth_vhost.c\n@@ -364,7 +364,7 @@ vhost_update_packet_xstats(struct vhost_queue *vq,\n \tuint64_t index;\n \tstruct vhost_stats *pstats = &vq->stats;\n \n-\tfor (i = 0; i < count ; i++) {\n+\tfor (i = 0; i < count; i++) {\n \t\tpkt_len = bufs[i]->pkt_len;\n \t\tif (pkt_len == 64) {\n \t\t\tpstats->xstats[VHOST_64_PKT]++;\ndiff --git a/drivers/net/virtio/virtio_rxtx.c b/drivers/net/virtio/virtio_rxtx.c\nindex fbc96df..1eb1fea 100644\n--- a/drivers/net/virtio/virtio_rxtx.c\n+++ b/drivers/net/virtio/virtio_rxtx.c\n@@ -125,7 +125,7 @@ virtqueue_dequeue_burst_rx(struct virtqueue *vq, struct rte_mbuf **rx_pkts,\n \tuint16_t i;\n \n \t/* Caller does the check */\n-\tfor (i = 0; i < num ; i++) {\n+\tfor (i = 0; i < num; i++) {\n \t\tused_idx = (uint16_t)(vq->vq_used_cons_idx & (vq->vq_nentries - 1));\n \t\tuep = &vq->vq_ring.used->ring[used_idx];\n \t\tdesc_idx = (uint16_t) uep->id;\n@@ -756,7 +756,7 @@ virtio_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)\n \thdr_size = hw->vtnet_hdr_size;\n \toffload = rx_offload_enabled(hw);\n \n-\tfor (i = 0; i < num ; i++) {\n+\tfor (i = 0; i < num; i++) {\n \t\trxm = rcv_pkts[i];\n \n \t\tPMD_RX_LOG(DEBUG, \"packet len:%d\", len[i]);\ndiff --git a/drivers/net/xenvirt/rte_eth_xenvirt.c b/drivers/net/xenvirt/rte_eth_xenvirt.c\nindex e404b77..e5e2eed 100644\n--- a/drivers/net/xenvirt/rte_eth_xenvirt.c\n+++ b/drivers/net/xenvirt/rte_eth_xenvirt.c\n@@ -99,7 +99,7 @@ eth_xenvirt_rx(void *q, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)\n \n \tnum = virtqueue_dequeue_burst(rxvq, rx_pkts, len, num);\n \tPMD_RX_LOG(DEBUG, \"used:%d dequeue:%d\\n\", nb_used, num);\n-\tfor (i = 0; i < num ; i ++) {\n+\tfor (i = 0; i < num; i++) {\n \t\trxm = rx_pkts[i];\n \t\tPMD_RX_LOG(DEBUG, \"packet len:%d\\n\", len[i]);\n \t\trxm->next = NULL;\n@@ -148,7 +148,7 @@ eth_xenvirt_tx(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)\n \tnum = (uint16_t)(likely(nb_used <= VIRTIO_MBUF_BURST_SZ) ? nb_used : VIRTIO_MBUF_BURST_SZ);\n \tnum = virtqueue_dequeue_burst(txvq, snd_pkts, len, num);\n \n-\tfor (i = 0; i < num ; i ++) {\n+\tfor (i = 0; i < num; i++) {\n \t\t/* mergable not supported, one segment only */\n \t\trte_pktmbuf_free_seg(snd_pkts[i]);\n \t}\ndiff --git a/drivers/net/xenvirt/rte_xen_lib.c b/drivers/net/xenvirt/rte_xen_lib.c\nindex 6c9a1d4..4524b3b 100644\n--- a/drivers/net/xenvirt/rte_xen_lib.c\n+++ b/drivers/net/xenvirt/rte_xen_lib.c\n@@ -402,7 +402,7 @@ grant_node_create(uint32_t pg_num, uint32_t *gref_arr, phys_addr_t *pa_arr, char\n \t\tif (rv == -1)\n \t\t\tbreak;\n \n-\t\tfor (i = 0; i < entries_per_pg && j < pg_num ; i++) {\n+\t\tfor (i = 0; i < entries_per_pg && j < pg_num; i++) {\n \t\t\t((struct grant_node_item *)ptr)->gref = gref_arr[j];\n \t\t\t((struct grant_node_item *)ptr)->pfn = pa_arr[j] >> pg_shift;\n \t\t\tptr = RTE_PTR_ADD(ptr, sizeof(struct grant_node_item));\ndiff --git a/drivers/net/xenvirt/virtqueue.h b/drivers/net/xenvirt/virtqueue.h\nindex 1bb6877..9473eca 100644\n--- a/drivers/net/xenvirt/virtqueue.h\n+++ b/drivers/net/xenvirt/virtqueue.h\n@@ -250,7 +250,7 @@ virtqueue_dequeue_burst(struct virtqueue *vq, struct rte_mbuf **rx_pkts, uint32_\n \tuint16_t used_idx, desc_idx;\n \tuint16_t i;\n \t/* Caller does the check */\n-\tfor (i = 0; i < num ; i ++) {\n+\tfor (i = 0; i < num; i++) {\n \t\tused_idx = (uint16_t)(vq->vq_used_cons_idx & (vq->vq_nentries - 1));\n \t\tuep = &vq->vq_ring.used->ring[used_idx];\n \t\tdesc_idx = (uint16_t) uep->id;\ndiff --git a/examples/ip_pipeline/cpu_core_map.c b/examples/ip_pipeline/cpu_core_map.c\nindex dd8f678..38c5786 100644\n--- a/examples/ip_pipeline/cpu_core_map.c\n+++ b/examples/ip_pipeline/cpu_core_map.c\n@@ -141,7 +141,7 @@ cpu_core_map_compute_eal(struct cpu_core_map *map)\n \n \t\tcore_id_contig = 0;\n \n-\t\tfor (core_id = 0; n_detected ; core_id++) {\n+\t\tfor (core_id = 0; n_detected; core_id++) {\n \t\t\tht_id = 0;\n \n \t\t\tfor (lcore_id = 0;\n@@ -362,7 +362,7 @@ cpu_core_map_compute_linux(struct cpu_core_map *map)\n \n \t\tcore_id_contig = 0;\n \n-\t\tfor (core_id = 0; n_detected ; core_id++) {\n+\t\tfor (core_id = 0; n_detected; core_id++) {\n \t\t\tht_id = 0;\n \n \t\t\tfor (lcore_id = 0; lcore_id < n_lcores; lcore_id++) {\ndiff --git a/examples/multi_process/l2fwd_fork/main.c b/examples/multi_process/l2fwd_fork/main.c\nindex f8a626b..c98b719 100644\n--- a/examples/multi_process/l2fwd_fork/main.c\n+++ b/examples/multi_process/l2fwd_fork/main.c\n@@ -307,7 +307,7 @@ l2fwd_malloc_shared_struct(void)\n \t\tif (mapping_id == NULL)\n \t\t\treturn -1;\n \n-\t\tfor (i = 0 ;i < RTE_MAX_LCORE; i++)\n+\t\tfor (i = 0; i < RTE_MAX_LCORE; i++)\n \t\t\tmapping_id[i] = INVALID_MAPPING_ID;\n \t}\n \treturn 0;\ndiff --git a/examples/netmap_compat/lib/compat_netmap.c b/examples/netmap_compat/lib/compat_netmap.c\nindex af2d9f3..fc1a32b 100644\n--- a/examples/netmap_compat/lib/compat_netmap.c\n+++ b/examples/netmap_compat/lib/compat_netmap.c\n@@ -485,7 +485,7 @@ rx_sync_ring(struct netmap_ring *ring, uint8_t port, uint16_t ring_number,\n \t\t\treturn -1;\n \n \t\t/* Put those n_rx packets in the Netmap structures */\n-\t\tfor (i = 0; i < n_rx ; i++) {\n+\t\tfor (i = 0; i < n_rx; i++) {\n \t\t\tmbuf_to_slot(rx_mbufs[i], ring, cur_slot);\n \t\t\trte_pktmbuf_free(rx_mbufs[i]);\n \t\t\tcur_slot = NETMAP_RING_NEXT(ring, cur_slot);\ndiff --git a/examples/performance-thread/l3fwd-thread/main.c b/examples/performance-thread/l3fwd-thread/main.c\nindex 7954b97..eb4f6e5 100644\n--- a/examples/performance-thread/l3fwd-thread/main.c\n+++ b/examples/performance-thread/l3fwd-thread/main.c\n@@ -1767,7 +1767,7 @@ process_burst(struct rte_mbuf *pkts_burst[MAX_PKT_BURST], int nb_rx,\n \t\t\t\tl3fwd_simple_forward(pkts_burst[j+7], portid);\n \t\t\t}\n \t\t}\n-\t\tfor (; j < nb_rx ; j++)\n+\t\tfor (; j < nb_rx; j++)\n \t\t\tl3fwd_simple_forward(pkts_burst[j], portid);\n \t}\n #elif (APP_LOOKUP_METHOD == APP_LOOKUP_LPM)\ndiff --git a/examples/qos_sched/app_thread.c b/examples/qos_sched/app_thread.c\nindex 15f117f..a3cdceb 100644\n--- a/examples/qos_sched/app_thread.c\n+++ b/examples/qos_sched/app_thread.c\n@@ -72,7 +72,7 @@ get_pkt_sched(struct rte_mbuf *m, uint32_t *subport, uint32_t *pipe,\n \t*traffic_class = (pdata[QUEUE_OFFSET] & 0x0F) &\n \t\t\t(RTE_SCHED_TRAFFIC_CLASSES_PER_PIPE - 1); /* Destination IP */\n \t*queue = ((pdata[QUEUE_OFFSET] >> 8) & 0x0F) &\n-\t\t\t(RTE_SCHED_QUEUES_PER_TRAFFIC_CLASS - 1) ; /* Destination IP */\n+\t\t\t(RTE_SCHED_QUEUES_PER_TRAFFIC_CLASS - 1); /* Destination IP */\n \t*color = pdata[COLOR_OFFSET] & 0x03; \t/* Destination IP */\n \n \treturn 0;\ndiff --git a/examples/quota_watermark/qw/main.c b/examples/quota_watermark/qw/main.c\nindex d4fcfde..2785d96 100644\n--- a/examples/quota_watermark/qw/main.c\n+++ b/examples/quota_watermark/qw/main.c\n@@ -367,7 +367,7 @@ main(int argc, char **argv)\n \t * Start pipeline_connect() on all the available slave lcores\n \t * but the last\n \t */\n-\tfor (lcore_id = 0 ; lcore_id < last_lcore_id; lcore_id++) {\n+\tfor (lcore_id = 0; lcore_id < last_lcore_id; lcore_id++) {\n \t\tif (rte_lcore_is_enabled(lcore_id) &&\n \t\t\t\tlcore_id != master_lcore_id) {\n \ndiff --git a/examples/vhost_xen/xenstore_parse.c b/examples/vhost_xen/xenstore_parse.c\nindex ab089f1..9e2ff81 100644\n--- a/examples/vhost_xen/xenstore_parse.c\n+++ b/examples/vhost_xen/xenstore_parse.c\n@@ -516,9 +516,8 @@ parse_mempoolnode(struct xen_guest *guest)\n err:\n \tif (gntnode)\n \t\txen_free_gntnode(gntnode);\n-\tfor (i = 0; i < MAX_XENVIRT_MEMPOOL ; i++) {\n+\tfor (i = 0; i < MAX_XENVIRT_MEMPOOL; i++)\n \t\tcleanup_mempool(&guest->mempool[i]);\n-\t}\n \t/* reinitialise mempool */\n \tbzero(&guest->mempool, MAX_XENVIRT_MEMPOOL * sizeof(guest->mempool[0]));\n \treturn -1;\ndiff --git a/lib/librte_distributor/rte_distributor.c b/lib/librte_distributor/rte_distributor.c\nindex 20ba9ff..4c09e25 100644\n--- a/lib/librte_distributor/rte_distributor.c\n+++ b/lib/librte_distributor/rte_distributor.c\n@@ -257,13 +257,13 @@ find_match_scalar(struct rte_distributor *d,\n \t * 4. Add any matches to the output\n \t */\n \n-\tfor (j = 0 ; j < RTE_DIST_BURST_SIZE; j++)\n+\tfor (j = 0; j < RTE_DIST_BURST_SIZE; j++)\n \t\toutput_ptr[j] = 0;\n \n \tfor (i = 0; i < d->num_workers; i++) {\n \t\tbl = &d->backlog[i];\n \n-\t\tfor (j = 0; j < RTE_DIST_BURST_SIZE ; j++)\n+\t\tfor (j = 0; j < RTE_DIST_BURST_SIZE; j++)\n \t\t\tfor (w = 0; w < RTE_DIST_BURST_SIZE; w++)\n \t\t\t\tif (d->in_flight_tags[i][j] == data_ptr[w]) {\n \t\t\t\t\toutput_ptr[j] = i+1;\n@@ -347,7 +347,7 @@ release(struct rte_distributor *d, unsigned int wkr)\n \t\td->in_flight_tags[wkr][i] = d->backlog[wkr].tags[i];\n \t}\n \tbuf->count = i;\n-\tfor ( ; i < RTE_DIST_BURST_SIZE ; i++) {\n+\tfor ( ; i < RTE_DIST_BURST_SIZE; i++) {\n \t\tbuf->bufptr64[i] = RTE_DISTRIB_GET_BUF;\n \t\td->in_flight_tags[wkr][i] = 0;\n \t}\n@@ -381,7 +381,7 @@ rte_distributor_process_v1705(struct rte_distributor *d,\n \n \tif (unlikely(num_mbufs == 0)) {\n \t\t/* Flush out all non-full cache-lines to workers. */\n-\t\tfor (wid = 0 ; wid < d->num_workers; wid++) {\n+\t\tfor (wid = 0; wid < d->num_workers; wid++) {\n \t\t\tif ((d->bufs[wid].bufptr64[0] & RTE_DISTRIB_GET_BUF)) {\n \t\t\t\trelease(d, wid);\n \t\t\t\thandle_returns(d, wid);\n@@ -491,7 +491,7 @@ rte_distributor_process_v1705(struct rte_distributor *d,\n \t}\n \n \t/* Flush out all non-full cache-lines to workers. */\n-\tfor (wid = 0 ; wid < d->num_workers; wid++)\n+\tfor (wid = 0; wid < d->num_workers; wid++)\n \t\tif ((d->bufs[wid].bufptr64[0] & RTE_DISTRIB_GET_BUF))\n \t\t\trelease(d, wid);\n \n@@ -666,7 +666,7 @@ rte_distributor_create_v1705(const char *name,\n \t * Set up the backog tags so they're pointing at the second cache\n \t * line for performance during flow matching\n \t */\n-\tfor (i = 0 ; i < num_workers ; i++)\n+\tfor (i = 0; i < num_workers; i++)\n \t\td->backlog[i].tags = &d->in_flight_tags[i][RTE_DIST_BURST_SIZE];\n \n \tdist_burst_list = RTE_TAILQ_CAST(rte_dist_burst_tailq.head,\ndiff --git a/lib/librte_eal/linuxapp/eal/eal_memory.c b/lib/librte_eal/linuxapp/eal/eal_memory.c\nindex daead31..6b41341 100644\n--- a/lib/librte_eal/linuxapp/eal/eal_memory.c\n+++ b/lib/librte_eal/linuxapp/eal/eal_memory.c\n@@ -470,7 +470,7 @@ map_all_hugepages(struct hugepage_file *hugepg_tbl, struct hugepage_info *hpi,\n \t\t\t/* reserve a virtual area for next contiguous\n \t\t\t * physical block: count the number of\n \t\t\t * contiguous physical pages. */\n-\t\t\tfor (j = i+1; j < hpi->num_pages[0] ; j++) {\n+\t\t\tfor (j = i+1; j < hpi->num_pages[0]; j++) {\n #ifdef RTE_ARCH_PPC_64\n \t\t\t\t/* The physical addresses are sorted in\n \t\t\t\t * descending order on PPC64 */\ndiff --git a/lib/librte_eal/linuxapp/eal/eal_xen_memory.c b/lib/librte_eal/linuxapp/eal/eal_xen_memory.c\nindex 19db1cb..1549c69 100644\n--- a/lib/librte_eal/linuxapp/eal/eal_xen_memory.c\n+++ b/lib/librte_eal/linuxapp/eal/eal_xen_memory.c\n@@ -287,7 +287,7 @@ rte_xen_dom0_memory_init(void)\n \n \t\tmemseg[memseg_idx].addr = vir_addr;\n \t\tmemseg[memseg_idx].phys_addr = page_size *\n-\t\t\tseginfo[memseg_idx].pfn ;\n+\t\t\tseginfo[memseg_idx].pfn;\n \t\tmemseg[memseg_idx].len = seginfo[memseg_idx].size;\n \t\tfor ( i = 0; i < seginfo[memseg_idx].size / RTE_PGSIZE_2M; i++)\n \t\t\tmemseg[memseg_idx].mfn[i] = seginfo[memseg_idx].mfn[i];\ndiff --git a/lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c b/lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c\nindex 99338c5..1812830 100644\n--- a/lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c\n+++ b/lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c\n@@ -1989,7 +1989,7 @@ void igb_reset(struct igb_adapter *adapter)\n \t\t * Clear all flags except indication that the PF has set\n \t\t * the VF MAC addresses administratively\n \t\t */\n-\t\tfor (i = 0 ; i < adapter->vfs_allocated_count; i++)\n+\t\tfor (i = 0; i < adapter->vfs_allocated_count; i++)\n \t\t\tadapter->vf_data[i].flags &= IGB_VF_FLAG_PF_SET_MAC;\n \n \t\t/* ping all the active vfs to let them know we are going down */\n@@ -6140,7 +6140,7 @@ static void igb_ping_all_vfs(struct igb_adapter *adapter)\n \tu32 ping;\n \tint i;\n \n-\tfor (i = 0 ; i < adapter->vfs_allocated_count; i++) {\n+\tfor (i = 0; i < adapter->vfs_allocated_count; i++) {\n \t\tping = E1000_PF_CONTROL_MSG;\n \t\tif (adapter->vf_data[i].flags & IGB_VF_FLAG_CTS)\n \t\t\tping |= E1000_VT_MSGTYPE_CTS;\ndiff --git a/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c b/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c\nindex d26016c..3a018c3 100644\n--- a/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c\n+++ b/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c\n@@ -1033,7 +1033,7 @@ void ixgbe_down(struct ixgbe_adapter *adapter)\n \t\tIXGBE_WRITE_REG(&adapter->hw, IXGBE_EITRSEL, 0);\n \n \t\t/* Mark all the VFs as inactive */\n-\t\tfor (i = 0 ; i < adapter->num_vfs; i++)\n+\t\tfor (i = 0; i < adapter->num_vfs; i++)\n \t\t\tadapter->vfinfo[i].clear_to_send = 0;\n \n \t\t/* ping all the active vfs to let them know we are going down */\ndiff --git a/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/kcompat.c b/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/kcompat.c\nindex d84c7cc..1aa7899 100644\n--- a/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/kcompat.c\n+++ b/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/kcompat.c\n@@ -168,7 +168,7 @@ int _kc_vsnprintf(char *buf, size_t size, const char *fmt, va_list args)\n \t\tsize = end - buf + 1;\n \t}\n \n-\tfor (; *fmt ; ++fmt) {\n+\tfor (; *fmt; ++fmt) {\n \t\tif (*fmt != '%') {\n \t\t\tif (str <= end)\n \t\t\t\t*str = *fmt;\ndiff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c\nindex a1b7447..937f30b 100644\n--- a/lib/librte_ether/rte_ethdev.c\n+++ b/lib/librte_ether/rte_ethdev.c\n@@ -1670,7 +1670,7 @@ rte_eth_xstats_get_by_id(uint8_t port_id, const uint64_t *ids, uint64_t *values,\n \t\t\t\treturn xcount;\n \n \t\t\tif (values != NULL)\n-\t\t\t\tfor (i = 0 ; i < (unsigned int)xcount; i++)\n+\t\t\t\tfor (i = 0; i < (unsigned int)xcount; i++)\n \t\t\t\t\tvalues[i + count] = xstats[i].value;\n \t\t}\n \ndiff --git a/lib/librte_sched/rte_approx.c b/lib/librte_sched/rte_approx.c\nindex 739f37d..ff6bdf9 100644\n--- a/lib/librte_sched/rte_approx.c\n+++ b/lib/librte_sched/rte_approx.c\n@@ -132,14 +132,14 @@ find_best_rational_approximation(uint32_t alpha_num, uint32_t d_num, uint32_t de\n \t\t}\n \n \t\t/* update the interval */\n-\t\tnew_p_a = p_b + (x - 1) * p_a ;\n+\t\tnew_p_a = p_b + (x - 1) * p_a;\n \t\tnew_q_a = q_b + (x - 1) * q_a;\n-\t\tnew_p_b = p_b + x * p_a ;\n+\t\tnew_p_b = p_b + x * p_a;\n \t\tnew_q_b = q_b + x * q_a;\n \n-\t\tp_a = new_p_a ;\n+\t\tp_a = new_p_a;\n \t\tq_a = new_q_a;\n-\t\tp_b = new_p_b ;\n+\t\tp_b = new_p_b;\n \t\tq_b = new_q_b;\n \n \t\t/* compute the number of steps to the right */\ndiff --git a/lib/librte_sched/rte_bitmap.h b/lib/librte_sched/rte_bitmap.h\nindex 010d752..e487b58 100644\n--- a/lib/librte_sched/rte_bitmap.h\n+++ b/lib/librte_sched/rte_bitmap.h\n@@ -500,7 +500,8 @@ __rte_bitmap_scan_read(struct rte_bitmap *bmp, uint32_t *pos, uint64_t *slab)\n \tuint64_t *slab2;\n \n \tslab2 = bmp->array2 + bmp->index2;\n-\tfor ( ; bmp->go2 ; bmp->index2 ++, slab2 ++, bmp->go2 = bmp->index2 & RTE_BITMAP_CL_SLAB_MASK) {\n+\tfor ( ; bmp->go2; bmp->index2++, slab2++,\n+\t\t\t bmp->go2 = bmp->index2 & RTE_BITMAP_CL_SLAB_MASK) {\n \t\tif (*slab2) {\n \t\t\t*pos = bmp->index2 << RTE_BITMAP_SLAB_BIT_SIZE_LOG2;\n \t\t\t*slab = *slab2;\ndiff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c\nindex b068609..12dabc2 100644\n--- a/test/test/test_cryptodev.c\n+++ b/test/test/test_cryptodev.c\n@@ -463,7 +463,7 @@ ut_setup(void)\n \t\t\t\"Failed to configure cryptodev %u\",\n \t\t\tts_params->valid_devs[0]);\n \n-\tfor (qp_id = 0; qp_id < ts_params->conf.nb_queue_pairs ; qp_id++) {\n+\tfor (qp_id = 0; qp_id < ts_params->conf.nb_queue_pairs; qp_id++) {\n \t\tTEST_ASSERT_SUCCESS(rte_cryptodev_queue_pair_setup(\n \t\t\tts_params->valid_devs[0], qp_id,\n \t\t\t&ts_params->qp_conf,\ndiff --git a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c\nindex 8cddaa7..4200611 100644\n--- a/test/test/test_cryptodev_perf.c\n+++ b/test/test/test_cryptodev_perf.c\n@@ -428,7 +428,7 @@ testsuite_setup(void)\n \t\t\tts_params->dev_id);\n \n \tts_params->qp_conf.nb_descriptors = PERF_NUM_OPS_INFLIGHT;\n-\tfor (qp_id = 0; qp_id < ts_params->conf.nb_queue_pairs ; qp_id++) {\n+\tfor (qp_id = 0; qp_id < ts_params->conf.nb_queue_pairs; qp_id++) {\n \n \t\tTEST_ASSERT_SUCCESS(rte_cryptodev_queue_pair_setup(\n \t\t\tts_params->dev_id, qp_id,\n@@ -1983,7 +1983,7 @@ test_perf_crypto_qp_vary_burst_size(uint16_t dev_num)\n \tTEST_ASSERT_NOT_NULL(test_crypto_session, \"Session creation failed\");\n \n \t/* Generate Crypto op data structure(s) */\n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\tstruct rte_mbuf *m = setup_test_string(ts_params->mbuf_mp,\n \t\t\t\tdata_params[0].expected.ciphertext,\n \t\t\t\tdata_params[0].length, 0);\n@@ -2030,7 +2030,7 @@ test_perf_crypto_qp_vary_burst_size(uint16_t dev_num)\n \tprintf(\"\\nDev No\\tQP No\\tNum Sent\\tNum Received\\tTx/Rx burst\");\n \tprintf(\"\\tRetries (Device Busy)\\tAverage IA cycle cost \"\n \t\t\t\"(assuming 0 retries)\");\n-\tfor (i = 2; i <= 128 ; i *= 2) {\n+\tfor (i = 2; i <= 128; i *= 2) {\n \t\tnum_sent = 0;\n \t\tnum_received = 0;\n \t\tretries = 0;\n@@ -2087,7 +2087,7 @@ test_perf_crypto_qp_vary_burst_size(uint16_t dev_num)\n \t}\n \tprintf(\"\\n\");\n \n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\trte_pktmbuf_free(c_ops[i]->sym->m_src);\n \t\trte_crypto_op_free(c_ops[i]);\n \t}\n@@ -2122,7 +2122,7 @@ test_perf_snow3G_optimise_cyclecount(struct perf_test_params *pparams)\n \tTEST_ASSERT_NOT_NULL(sess, \"Session creation failed\");\n \n \t/* Generate Crypto op data structure(s)*/\n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\tstruct rte_mbuf *m = test_perf_create_pktmbuf(\n \t\t\t\t\t\tts_params->mbuf_mp,\n \t\t\t\t\t\tpparams->buf_size);\n@@ -2159,7 +2159,7 @@ test_perf_snow3G_optimise_cyclecount(struct perf_test_params *pparams)\n \tprintf(\"\\nOps Tx\\tOps Rx\\tOps/burst \");\n \tprintf(\"Retries EmptyPolls\\tIACycles/CyOp\\tIACycles/Burst\\tIACycles/Byte\");\n \n-\tfor (i = 2; i <= 128 ; i *= 2) {\n+\tfor (i = 2; i <= 128; i *= 2) {\n \t\tnum_sent = 0;\n \t\tnum_ops_received = 0;\n \t\tretries = 0;\n@@ -2218,7 +2218,7 @@ test_perf_snow3G_optimise_cyclecount(struct perf_test_params *pparams)\n \t}\n \tprintf(\"\\n\");\n \n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\trte_pktmbuf_free(c_ops[i]->sym->m_src);\n \t\trte_crypto_op_free(c_ops[i]);\n \t}\n@@ -2314,7 +2314,7 @@ test_perf_openssl_optimise_cyclecount(struct perf_test_params *pparams)\n \tTEST_ASSERT_NOT_NULL(sess, \"Session creation failed\");\n \n \t/* Generate Crypto op data structure(s)*/\n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\tstruct rte_mbuf *m = test_perf_create_pktmbuf(\n \t\t\t\t\t\tts_params->mbuf_mp,\n \t\t\t\t\t\tpparams->buf_size);\n@@ -2373,7 +2373,7 @@ test_perf_openssl_optimise_cyclecount(struct perf_test_params *pparams)\n \tprintf(\"Retries EmptyPolls\\tIACycles/CyOp\\tIACycles/Burst\\t\"\n \t\t\t\"IACycles/Byte\");\n \n-\tfor (i = 2; i <= 128 ; i *= 2) {\n+\tfor (i = 2; i <= 128; i *= 2) {\n \t\tnum_sent = 0;\n \t\tnum_ops_received = 0;\n \t\tretries = 0;\n@@ -2438,7 +2438,7 @@ test_perf_openssl_optimise_cyclecount(struct perf_test_params *pparams)\n \t}\n \tprintf(\"\\n\");\n \n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\trte_pktmbuf_free(c_ops[i]->sym->m_src);\n \t\trte_crypto_op_free(c_ops[i]);\n \t}\n@@ -2479,7 +2479,7 @@ test_perf_armv8_optimise_cyclecount(struct perf_test_params *pparams)\n \t\tsess = NULL;\n \n \t/* Generate Crypto op data structure(s)*/\n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\tstruct rte_mbuf *m = test_perf_create_pktmbuf(\n \t\t\t\t\t\tts_params->mbuf_mp,\n \t\t\t\t\t\tpparams->buf_size);\n@@ -2510,7 +2510,7 @@ test_perf_armv8_optimise_cyclecount(struct perf_test_params *pparams)\n \tprintf(\"Retries \"\n \t\t\"EmptyPolls\\tIACycles/CyOp\\tIACycles/Burst\\tIACycles/Byte\");\n \n-\tfor (i = 2; i <= 128 ; i *= 2) {\n+\tfor (i = 2; i <= 128; i *= 2) {\n \t\tnum_sent = 0;\n \t\tnum_ops_received = 0;\n \t\tretries = 0;\n@@ -2574,7 +2574,7 @@ test_perf_armv8_optimise_cyclecount(struct perf_test_params *pparams)\n \t}\n \tprintf(\"\\n\");\n \n-\tfor (i = 0; i < num_to_submit ; i++) {\n+\tfor (i = 0; i < num_to_submit; i++) {\n \t\trte_pktmbuf_free(c_ops[i]->sym->m_src);\n \t\trte_crypto_op_free(c_ops[i]);\n \t}\ndiff --git a/test/test/test_eventdev_sw.c b/test/test/test_eventdev_sw.c\nindex ba8c053..8e7829f 100644\n--- a/test/test/test_eventdev_sw.c\n+++ b/test/test/test_eventdev_sw.c\n@@ -2696,7 +2696,7 @@ parallel_basic(struct test *t, int check_order)\n \n \t/* Check to see if the sequence numbers are in expected order */\n \tif (check_order) {\n-\t\tfor (j = 0 ; j < deq_pkts ; j++) {\n+\t\tfor (j = 0; j < deq_pkts; j++) {\n \t\t\tif (deq_ev[j].mbuf->seqn != MAGIC_SEQN + j) {\n \t\t\t\tprintf(\n \t\t\t\t\t\"%d: Incorrect sequence number(%d) from port %d\\n\",\ndiff --git a/test/test/test_malloc.c b/test/test/test_malloc.c\nindex 013fd44..d813d4d 100644\n--- a/test/test/test_malloc.c\n+++ b/test/test/test_malloc.c\n@@ -106,7 +106,7 @@ test_align_overlap_per_lcore(__attribute__((unused)) void *arg)\n \t\t\tret = -1;\n \t\t\tbreak;\n \t\t}\n-\t\tfor(j = 0; j < 1000 ; j++) {\n+\t\tfor (j = 0; j < 1000; j++) {\n \t\t\tif( *(char *)p1 != 0) {\n \t\t\t\tprintf(\"rte_zmalloc didn't zero\"\n \t\t\t\t \"the allocated memory\\n\");\n@@ -178,7 +178,7 @@ test_reordered_free_per_lcore(__attribute__((unused)) void *arg)\n \t\t\tret = -1;\n \t\t\tbreak;\n \t\t}\n-\t\tfor(j = 0; j < 1000 ; j++) {\n+\t\tfor (j = 0; j < 1000; j++) {\n \t\t\tif( *(char *)p1 != 0) {\n \t\t\t\tprintf(\"rte_zmalloc didn't zero\"\n \t\t\t\t \"the allocated memory\\n\");\ndiff --git a/test/test/test_memory.c b/test/test/test_memory.c\nindex 921bdc8..8e01f0f 100644\n--- a/test/test/test_memory.c\n+++ b/test/test/test_memory.c\n@@ -75,7 +75,7 @@ test_memory(void)\n \n \t/* try to read memory (should not segfault) */\n \tmem = rte_eal_get_physmem_layout();\n-\tfor (i = 0; i < RTE_MAX_MEMSEG && mem[i].addr != NULL ; i++) {\n+\tfor (i = 0; i < RTE_MAX_MEMSEG && mem[i].addr != NULL; i++) {\n \n \t\t/* check memory */\n \t\tfor (j = 0; j<mem[i].len; j++) {\ndiff --git a/test/test/test_mempool.c b/test/test/test_mempool.c\nindex 0a44239..4284dbc 100644\n--- a/test/test/test_mempool.c\n+++ b/test/test/test_mempool.c\n@@ -329,7 +329,7 @@ my_mp_init(struct rte_mempool *mp, __attribute__((unused)) void *arg)\n {\n \tprintf(\"mempool name is %s\\n\", mp->name);\n \t/* nothing to be implemented here*/\n-\treturn ;\n+\treturn;\n }\n \n /*\ndiff --git a/test/test/test_ring.c b/test/test/test_ring.c\nindex d664b04..5adc13c 100644\n--- a/test/test/test_ring.c\n+++ b/test/test/test_ring.c\n@@ -155,9 +155,8 @@ test_ring_basic(void)\n \tif (src == NULL)\n \t\tgoto fail;\n \n-\tfor (i = 0; i < RING_SIZE*2 ; i++) {\n+\tfor (i = 0; i < RING_SIZE*2; i++)\n \t\tsrc[i] = (void *)(unsigned long)i;\n-\t}\n \tcur_src = src;\n \n \t/* alloc some room for copied objects */\n@@ -358,9 +357,8 @@ test_ring_burst_basic(void)\n \tif (src == NULL)\n \t\tgoto fail;\n \n-\tfor (i = 0; i < RING_SIZE*2 ; i++) {\n+\tfor (i = 0; i < RING_SIZE*2; i++)\n \t\tsrc[i] = (void *)(unsigned long)i;\n-\t}\n \tcur_src = src;\n \n \t/* alloc some room for copied objects */\ndiff --git a/test/test/test_table_acl.c b/test/test/test_table_acl.c\nindex 08c100f..fefcbfa 100644\n--- a/test/test/test_table_acl.c\n+++ b/test/test/test_table_acl.c\n@@ -650,7 +650,7 @@ setup_acl_pipeline(void)\n \t}\n \n \t/* Enable input ports */\n-\tfor (i = 0; i < N_PORTS ; i++)\n+\tfor (i = 0; i < N_PORTS; i++)\n \t\tif (rte_pipeline_port_in_enable(p, port_in_id[i]))\n \t\t\trte_panic(\"Unable to enable input port %u\\n\",\n \t\t\t\tport_in_id[i]);\ndiff --git a/test/test/test_table_pipeline.c b/test/test/test_table_pipeline.c\nindex a6fef72..121caa3 100644\n--- a/test/test/test_table_pipeline.c\n+++ b/test/test/test_table_pipeline.c\n@@ -410,7 +410,7 @@ setup_pipeline(int test_type)\n \t}\n \n \t/* Enable input ports */\n-\tfor (i = 0; i < N_PORTS ; i++)\n+\tfor (i = 0; i < N_PORTS; i++)\n \t\tif (rte_pipeline_port_in_enable(p, port_in_id[i]))\n \t\t\trte_panic(\"Unable to enable input port %u\\n\",\n \t\t\t\tport_in_id[i]);\n", "prefixes": [ "dpdk-dev" ] }