On Mon, Nov 19, 2007 at 10:40:12PM -0800, Joe Perches wrote: > Change LOOPBACK MULTICAST LOCAL_MCAST BADCLASS and ZERONET > macros to inline functions ipv4_is_<type>(__be32 addr) > > Adds some type safety and arguably some readability > > No change in compiled image size > > Signed-off-by: Joe Perches <[EMAIL PROTECTED]>
I can understand the motivation for moving the functionality into inline functions in order to get some type safety. But what is the motivation for changing all the macro calls to function calls, when you have set up the marcos to call the appropriate functions? Surely if you want to do that you should just ditch the marcos all together. Perhaps I am missing a patch? Also, as the function names are longer than the macro name you are creating lines that are > 80 columns wide that used not to be in that state. For my eyes that reduced readability (substantially). -- Horms - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
