Re: [B.A.T.M.A.N.] [PATCH 1/6] batman-adv: implement an helper function to forge unicast packets

On Mon, Oct 31, 2011 at 01:05:46AM +0100, Simon Wunderlich wrote:

On Sun, Oct 30, 2011 at 09:55:57AM +0100, Antonio Quartulli wrote:

...

-find_router: +prepare_packet: /** * find_router():

I guess the comment should be changed as well when the label is renamed.

Actually this comment refers to the function below, not to the label.

...

*  - if orig_node is NULL it returns NULL
*  - increases neigh_nodes refcount if found.
*/

neigh_node = find_router(bat_priv, orig_node, NULL);

Do we really need to remove this newline?

Don't think so ;)

Thanks,