Re: [PATCH net-next-2.6] net/bonding: adjust codingstyle for bond_3ad files

On Wed, 11 May 2011 19:52:21 -0300
Rafael Aquini [off-list ref] wrote:

Howdy,

While I was studying what bond_3ad has under its hood, I realized its coding
style did not follow all Documentation/CodingStyle recommendations. As a tiny
collaboration I did some mods there, in an attempt to make that code stick as
closely as possible with Kernel's coding style. Also, Nicolas Kaiser has kindly
suggested some conditional simplifications integrated in this patch.
Modifications:
        * switched all comments from C99-style to C89-style;
        * replaced MAC_ADDRESS_COMPARE macro for compare_ether_addr();
	* print info out on unexpected status checkings;
        * simplify conditionals:
		(a || (!a && !b)) => (a || !b)
		(!(!a && b)) => (a || !b)

Signed-off-by: Nicolas Kaiser <redacted>
Signed-off-by: Rafael Aquini <redacted>

This should be split into three separate patches.

Always assume your change will break something for somebody and it will
be necessary for some later investigation to bisect out the cause.



--