[ovs-discuss] [PATCH 2/3] vswitchd: Don't build non-used function to prevent compiler warnings

Justin Pettit jpettit at nicira.com
Wed Jul 29 22:52:54 UTC 2009


The function send_ofmp_error_msg() is not currently used, so it was
producing compiler warnings that it was defined.  This checkin #if 0's
it out, so that the file compiles cleanly.
---
 vswitchd/mgmt.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/vswitchd/mgmt.c b/vswitchd/mgmt.c
index 45c3580..9546b46 100644
--- a/vswitchd/mgmt.c
+++ b/vswitchd/mgmt.c
@@ -526,6 +526,7 @@ send_config_update_ack(uint32_t xid, bool success)
     send_openflow_buffer(buffer);
 }
 
+#if 0  /* Not currently used, so remove to prevent compiler warnings. */
 static void
 send_ofmp_error_msg(uint32_t xid, uint16_t type, uint16_t code, 
             const void *data, size_t len)
@@ -539,6 +540,7 @@ send_ofmp_error_msg(uint32_t xid, uint16_t type, uint16_t code,
     memcpy(oem->data, data, len);
     send_openflow_buffer(buffer);
 }
+#endif
 
 static void
 send_error_msg(uint32_t xid, uint16_t type, uint16_t code, 
-- 
1.6.3.1





More information about the discuss mailing list