vconn: Remove unnecessary forward declarations and #includes from hea…

…der. This required fix-ups in a few other files that accidentally depended upon vconn.h including those other headers.
hugb · Oct 4, 2011 · 3021ea6 · 3021ea6
1 parent b47e2a8
commit 3021ea6
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 9 deletions.
diff --git a/lib/vconn.h b/lib/vconn.h
@@ -17,23 +17,15 @@
 #ifndef VCONN_H
 #define VCONN_H 1
 
-#include <assert.h>
 #include <stdbool.h>
-#include <stddef.h>
-#include <stdint.h>
-
-#include "flow.h"
+#include "openvswitch/types.h"
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
 struct list;
 struct ofpbuf;
-struct ofp_action_header;
-struct ofp_header;
-struct ofp_match;
-struct ofp_stats_msg;
 struct pvconn;
 struct vconn;
 

diff --git a/ofproto/pinsched.c b/ofproto/pinsched.c
@@ -21,6 +21,7 @@
 #include <arpa/inet.h>
 #include <stdint.h>
 #include <stdlib.h>
+#include "hash.h"
 #include "hmap.h"
 #include "ofpbuf.h"
 #include "openflow/openflow.h"

diff --git a/tests/test-vconn.c b/tests/test-vconn.c
@@ -22,6 +22,7 @@
 #include <stdlib.h>
 #include <unistd.h>
 #include "command-line.h"
+#include "openflow/openflow.h"
 #include "poll-loop.h"
 #include "socket-util.h"
 #include "stream.h"

diff --git a/vswitchd/ovs-vswitchd.c b/vswitchd/ovs-vswitchd.c
@@ -34,6 +34,7 @@
 #include "dummy.h"
 #include "leak-checker.h"
 #include "netdev.h"
+#include "openflow/openflow.h"
 #include "ovsdb-idl.h"
 #include "poll-loop.h"
 #include "process.h"