Re: [PATCH] bridge: return boolean instead of integer in br_multicast_is_router

From: Gustavo A. R. Silva
Date: Thu Jan 18 2018 - 20:17:30 EST



Quoting Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx>:

On Thu, 18 Jan 2018 17:37:45 -0600
"Gustavo A. R. Silva" <garsilva@xxxxxxxxxxxxxx> wrote:

Return statements in functions returning bool should use
true/false instead of 1/0.

This issue was detected with the help of Coccinelle.

Signed-off-by: Gustavo A. R. Silva <garsilva@xxxxxxxxxxxxxx>

LGTM

Fixes: 85b352693264 ("bridge: Fix build error when IGMP_SNOOPING is not enabled")
Reviewed-by: Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx>

Thanks Stephen.
--
Gustavo