test/ring: remove unnecessary braces
authorHonnappa Nagarahalli <honnappa.nagarahalli@arm.com>
Sun, 25 Oct 2020 05:45:55 +0000 (00:45 -0500)
committerDavid Marchand <david.marchand@redhat.com>
Thu, 29 Oct 2020 13:13:31 +0000 (14:13 +0100)
Remove unnecessary braces to improve readability.

Signed-off-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
app/test/test_ring.c
app/test/test_ring.h

index a62cb26..2913e41 100644 (file)
@@ -247,7 +247,7 @@ static void**
 test_ring_inc_ptr(void **obj, int esize, unsigned int n)
 {
        /* Legacy queue APIs? */
-       if ((esize) == -1)
+       if (esize == -1)
                return ((void **)obj) + n;
        else
                return (void **)(((uint32_t *)obj) +
index d4b15af..2785559 100644 (file)
@@ -35,11 +35,11 @@ test_ring_create(const char *name, int esize, unsigned int count,
                int socket_id, unsigned int flags)
 {
        /* Legacy queue APIs? */
-       if ((esize) == -1)
-               return rte_ring_create((name), (count), (socket_id), (flags));
+       if (esize == -1)
+               return rte_ring_create(name, count, socket_id, flags);
        else
-               return rte_ring_create_elem((name), (esize), (count),
-                                               (socket_id), (flags));
+               return rte_ring_create_elem(name, esize, count,
+                                               socket_id, flags);
 }
 
 static __rte_always_inline unsigned int
@@ -47,7 +47,7 @@ test_ring_enqueue(struct rte_ring *r, void **obj, int esize, unsigned int n,
                        unsigned int api_type)
 {
        /* Legacy queue APIs? */
-       if ((esize) == -1)
+       if (esize == -1)
                switch (api_type) {
                case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_SINGLE):
                        return rte_ring_enqueue(r, *obj);
@@ -108,7 +108,7 @@ test_ring_dequeue(struct rte_ring *r, void **obj, int esize, unsigned int n,
                        unsigned int api_type)
 {
        /* Legacy queue APIs? */
-       if ((esize) == -1)
+       if (esize == -1)
                switch (api_type) {
                case (TEST_RING_THREAD_DEF | TEST_RING_ELEM_SINGLE):
                        return rte_ring_dequeue(r, obj);