GNU C++ isn't too happy with ovs-atomic.h.  We could fix that (maybe we
should) but the report I received from a C++ user implied to me that it
would be just as useful to just drop the unnecessary #include
"ovs-atomic.h" from hmap.h.

Reported-by: Michael Hu <[email protected]>
Signed-off-by: Ben Pfaff <[email protected]>
---
 lib/cfm.c          | 1 +
 lib/hmap.h         | 3 +--
 lib/lacp.c         | 3 ++-
 lib/netdev-dummy.c | 3 ++-
 lib/ovs-lldp.h     | 1 +
 5 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/lib/cfm.c b/lib/cfm.c
index 380c5b0..6a507a1 100644
--- a/lib/cfm.c
+++ b/lib/cfm.c
@@ -29,6 +29,7 @@
 #include "hash.h"
 #include "hmap.h"
 #include "netdev.h"
+#include "ovs-atomic.h"
 #include "packets.h"
 #include "poll-loop.h"
 #include "random.h"
diff --git a/lib/hmap.h b/lib/hmap.h
index 85241f0..dc1e85c 100644
--- a/lib/hmap.h
+++ b/lib/hmap.h
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2008, 2009, 2010, 2012, 2013 Nicira, Inc.
+ * Copyright (c) 2008, 2009, 2010, 2012, 2013, 2015 Nicira, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,7 +19,6 @@
 
 #include <stdbool.h>
 #include <stdlib.h>
-#include "ovs-atomic.h"
 #include "util.h"
 
 #ifdef  __cplusplus
diff --git a/lib/lacp.c b/lib/lacp.c
index 6535e6d..65149fd 100644
--- a/lib/lacp.c
+++ b/lib/lacp.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2011, 2012, 2013, 2014 Nicira, Inc.
+/* Copyright (c) 2011, 2012, 2013, 2014, 2015 Nicira, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -23,6 +23,7 @@
 #include "hash.h"
 #include "hmap.h"
 #include "dp-packet.h"
+#include "ovs-atomic.h"
 #include "packets.h"
 #include "poll-loop.h"
 #include "seq.h"
diff --git a/lib/netdev-dummy.c b/lib/netdev-dummy.c
index 7fb103e..24c91c2 100644
--- a/lib/netdev-dummy.c
+++ b/lib/netdev-dummy.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010, 2011, 2012, 2013 Nicira, Inc.
+ * Copyright (c) 2010, 2011, 2012, 2013, 2015 Nicira, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,6 +30,7 @@
 #include "odp-util.h"
 #include "ofp-print.h"
 #include "ofpbuf.h"
+#include "ovs-atomic.h"
 #include "packets.h"
 #include "pcap-file.h"
 #include "poll-loop.h"
diff --git a/lib/ovs-lldp.h b/lib/ovs-lldp.h
index 825096b..66288a5 100644
--- a/lib/ovs-lldp.h
+++ b/lib/ovs-lldp.h
@@ -24,6 +24,7 @@
 #include "hmap.h"
 #include "list.h"
 #include "lldp/lldpd.h"
+#include "ovs-atomic.h"
 #include "packets.h"
 #include "timer.h"
 
-- 
2.1.3

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to