bridge: Avoid 'statement with no effect' compiler warnings (0cb2bbbe) · Commits · 戴 / test

Instead of issuing (0) statements when !CONFIG_SYSFS which will cause 'warning: ', we'll use inline statements instead. This will effectively do the same thing, but suppress any unnecessary warnings. Cc: Stephen Hemminger <shemminger@vyatta.com> Cc: bridge@lists.linux-foundation.org Cc: netdev@vger.kernel.org Signed-off-by:

Lee Jones <lee.jones@linaro.org> Signed-off-by:

David S. Miller <davem@davemloft.net>

net/bridge/br_private.h

+4 −4

Original line number	Diff line number	Diff line
		@@ -570,10 +570,10 @@ extern void br_sysfs_delbr(struct net_device *dev);

		#else

		#define br_sysfs_addif(p) (0)
		#define br_sysfs_renameif(p) (0)
		#define br_sysfs_addbr(dev) (0)
		#define br_sysfs_delbr(dev) do { } while(0)
		static inline int br_sysfs_addif(struct net_bridge_port *p) { return 0; }
		static inline int br_sysfs_renameif(struct net_bridge_port *p) { return 0; }
		static inline int br_sysfs_addbr(struct net_device *dev) { return 0; }
		static inline void br_sysfs_delbr(struct net_device *dev) { return; }
		#endif /* CONFIG_SYSFS */

		#endif

Admin message