The OVS coding style document says that a .c file should include the
corresponding .h file first, to ensure that the .h file includes all of
its dependencies, but this file didn't do that.

Signed-off-by: Ben Pfaff <b...@ovn.org>
---
Depends on https://patchwork.ozlabs.org/patch/799445/.

 lib/ofp-print.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/ofp-print.c b/lib/ofp-print.c
index 16d617ab5008..f77b31baf0b4 100644
--- a/lib/ofp-print.c
+++ b/lib/ofp-print.c
@@ -16,6 +16,8 @@
 
 #include <config.h>
 
+#include "openvswitch/ofp-print.h"
+
 #include <errno.h>
 #include <inttypes.h>
 #include <sys/types.h>
@@ -43,7 +45,6 @@
 #include "openvswitch/ofp-actions.h"
 #include "openvswitch/ofp-errors.h"
 #include "openvswitch/ofp-msgs.h"
-#include "openvswitch/ofp-print.h"
 #include "openvswitch/ofp-util.h"
 #include "openvswitch/ofpbuf.h"
 #include "openvswitch/type-props.h"
-- 
2.10.2

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to