From: Ben Pfaff <blp@nicira.com>
Date: Tue, 4 Oct 2011 04:52:39 +0000 (-0700)
Subject: vconn: Remove unnecessary forward declarations and #includes from header.
X-Git-Tag: v1.3.0~153
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=3021ea6074bcc5a5bc7f1434ade97d84759e8d14;p=sliver-openvswitch.git

vconn: Remove unnecessary forward declarations and #includes from header.

This required fix-ups in a few other files that accidentally depended upon
vconn.h including those other headers.
---

diff --git a/lib/vconn.h b/lib/vconn.h
index 3c1ffe9c2..74b6b49cb 100644
--- a/lib/vconn.h
+++ b/lib/vconn.h
@@ -17,12 +17,8 @@
 #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" {
@@ -30,10 +26,6 @@ extern "C" {
 
 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
index 695d9b46a..6d7eb1301 100644
--- 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
index 2b14fa85d..42940c302 100644
--- 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
index 4910d699c..359e3ab1a 100644
--- 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"