>> -out: >> + batadv_tt_global_entry_free_ref(tt_global_entry); >> +local_entry_free: >> + batadv_tt_local_entry_free_ref(tt_local_entry); >> +vlan_free: >> batadv_softif_vlan_free_ref(vlan); >> - if (tt_global_entry) >> - batadv_tt_global_entry_free_ref(tt_global_entry); >> - if (tt_local_entry) >> - batadv_tt_local_entry_free_ref(tt_local_entry); >> return ret;
> if you really want to make this codestyle change, I'd suggest you to go > through the whole batman-adv code and apply the same change where needed. Thanks for your interest in similar source code changes. I would prefer general acceptance for this specific update suggestion before I might invest further software development efforts for the affected network module. > It does not make sense to change the codestyle in one spot only. I agree in the way that I would be nice if more places can still be improved. > On top of that, by going through the batman-adv code you might agree > that the current style is actually not a bad idea. I got the impression that the current Linux coding style convention disagrees around the affected jump label selection to some degree, doesn't it? Regards, Markus -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
