[ovs-dev] [PATCH] bridge: Remove superfluous "inline" from ofp12_controller_role_to_str().

Ben Pfaff blp at nicira.com
Thu Nov 21 00:50:22 UTC 2013


Putting "static inline" on a function definition in a .c file does not help
the compiler and does suppress warnings for unused functions that one would
want, should the function ever become unused.

Signed-off-by: Ben Pfaff <blp at nicira.com>
---
 vswitchd/bridge.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 6ce7d2b..33dc738 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -2118,7 +2118,7 @@ run_system_stats(void)
     }
 }
 
-static inline const char *
+static const char *
 ofp12_controller_role_to_str(enum ofp12_controller_role role)
 {
     switch (role) {
-- 
1.7.10.4




More information about the dev mailing list