On Wednesday 19 March 2014 18:55:37 Antonio Quartulli wrote: > There is no need to use a label and a goto for code that is > used once only. > Moreover having a goto for a single return statement should > always be avoided. > > Introduced by e368857f66620b8483166e8e6556d9c87f9b3e71 > ("batman-adv: Multicast Listener Announcements via Translation Table") > > Cc: Linus Lüssing > Signed-off-by: Antonio Quartulli > --- > > Changes from v3: > - simply has_bridge() function even more > > Changes from v2: > - added compat code change to 2/11 to accommodate new use of > netdev_master_upper_dev_get_rcu() > > Changes from v1: > - change "we are out of memory" to "in case of memory allocation failure" in > 3/11 > - fix commit message in 10/11 > > > multicast.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) Applied in revision 6f54268. Thanks, Marek