lkml.org 
[lkml]   [2015]   [Nov]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [B.A.T.M.A.N.] [PATCH 3/3] batman-adv: Less function calls in batadv_is_ap_isolated() after error detection
From
Date
>> -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


\
 
 \ /
  Last update: 2015-11-20 12:21    [W:1.003 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site