eal: remove return from functions declared void return type
Checks
Commit Message
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 <roretzla@linux.microsoft.com>
---
lib/eal/common/rte_malloc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
> From: Tyler Retzlaff [mailto:roretzla@linux.microsoft.com]
> 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 <roretzla@linux.microsoft.com>
> ---
Reviewed-by: Morten Brørup <mb@smartsharesystems.com>
Recheck-request: iol-broadcom-Functional
Sorry, this series was affected by an infra failure today.
On Tue, Oct 17, 2023 at 01:51:39PM -0400, Patrick Robb wrote:
> 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. :)
On Tue, Oct 17, 2023 at 3:15 PM Morten Brørup <mb@smartsharesystems.com> wrote:
>
> > From: Tyler Retzlaff [mailto:roretzla@linux.microsoft.com]
> > 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 <roretzla@linux.microsoft.com>
> Reviewed-by: Morten Brørup <mb@smartsharesystems.com>
Applied, thanks.
@@ -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 *