Skip to contentSkip to conversation

eal: remove return from functions declared void return type

PATCH Applied Tyler Retzlaff 4 participants 5 messages Oct 17, 2023 – Oct 26, 2023

lore
Share & tools
lore

Tyler Retzlaff

author
#0 lore raw

PATCH 1/1 eal: remove return from functions declared void return type

Remove return from rte_free and eal_free_no_trace both functions are declared to have a return type of void so they shouldn't return a value.

Signed-off-by Tyler Retzlaff <[email protected]>
---
 lib/eal/common/rte_malloc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
lib/eal/common/rte_malloc.c +2 -2
index ebafef3..9db0c39 100644
@@ -41,13 +41,13 @@
 void
 rte_free(void *addr)
 {
-	return mem_free(addr, true);
+	mem_free(addr, true);
 }
 
 void
 eal_free_no_trace(void *addr)
 {
-	return mem_free(addr, false);
+	mem_free(addr, false);
 }
 
 static void *
Signature · 1 line
1.8.3.1
#2 lore raw

Recheck-request: iol-broadcom-Functional

Sorry, this series was affected by an infra failure today.

author
#3 lore raw
Quoted 3 lines · Patrick Robb

Recheck-request: iol-broadcom-Functional

Sorry, this series was affected by an infra failure today.

No big deal, I haven't got a lot of concerns over the correctness of this particular series. :)

Applied
#4 lore raw
In reply to Morten Brørup
Quoted 9 lines · Morten Brørup
Quoted 8 lines

From: Tyler Retzlaff [mailto:[email protected]]
Sent: Tuesday, 17 October 2023 15.04

Remove return from rte_free and eal_free_no_trace both functions are declared to have a return type of void so they shouldn't return a value.

Signed-off-by Tyler Retzlaff <[email protected]>
Reviewed-by Morten Brørup <[email protected]>

Applied, thanks.

Signature · 1 line
David Marchand