On Wednesday 25 May 2016 23:27:31 Antonio Quartulli wrote:
The routing API data structure contains several function pointers that can easily be grouped together based on the component they work with.
Split the API in subobjects in order to improve definition readability.
At the same time, remove the "bat_" prefix from the API object and its fields names. These are batman-adv private structs and there is no need to always prepend such prefix, which only makes function invocations much much longer.
Signed-off-by: Antonio Quartulli a@unstable.cc
My eyes don't want to read diffs anymore which only removes bat_ and replaces a "_" with a ".". I want more interesting stuff happening in my movies^Wpatches :)
Reviewed-by: Sven Eckelmann sven@narfation.org
Btw. this is rather unrelated to your patch. But the kerneldoc contains "(optional)" statements. Following functions don't have it (and didn't have it before your patch):
* iface.activate * neigh.hardif_init * orig.free * orig.add_if * orig.del_if
Kind regards, Sven