On 11/21/2012 06:59 AM, Ján Tomko wrote:
bridge_driver.h: silence gcc warnings:
statement with no effect [-Wunused-value]
unused variable 'net' [-Wunused-variable]
virdrivermoduletest.c: don't require network driver module
if it hasn't been built.
---
src/network/bridge_driver.h | 4 ++--
tests/virdrivermoduletest.c | 8 ++++++++
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/network/bridge_driver.h b/src/network/bridge_driver.h
index 0fae275..1c42d81 100644
--- a/src/network/bridge_driver.h
+++ b/src/network/bridge_driver.h
@@ -53,8 +53,8 @@ int networkBuildDhcpDaemonCommandLine(virNetworkObjPtr network,
# else
/* Define no-op replacements that don't drag in any link dependencies. */
# define networkAllocateActualDevice(iface) 0
-# define networkNotifyActualDevice(iface) 0
-# define networkReleaseActualDevice(iface) 0
+# define networkNotifyActualDevice(iface) (iface=iface,0)
+# define networkReleaseActualDevice(iface) (iface=iface,0)
I would write this as (iface = iface, 0) for consistent operator spacing
with the rest of our code.
+++ b/tests/virdrivermoduletest.c
@@ -83,10 +83,18 @@ mymain(void)
TEST("interface", NULL);
#endif
#ifdef WITH_QEMU
+# ifdef WITH_NETWORK
TEST("qemu", "network");
+# else
+ TEST("qemu", NULL);
+# endif
#endif
#ifdef WITH_LXC
+# ifdef WITH_NETWORK
TEST("lxc", "network");
+# else
+ TEST("lxc", NULL);
+# endif
#endif
#ifdef WITH_UML
TEST("uml", NULL);
Is it any easier to do:
#ifdef WITH_NETWORK
# define USE_NETWORK "network"
#else
# define USE_NETWORK NULL
#endif
rather than repeated nested #ifdef? But that's cosmetic, so I don't
care if you don't change it.
ACK with the formatting nit fixed.
--
Eric Blake eblake(a)redhat.com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org