Get rid of shadowed booleans

There are still two places where we are using 1bit width unsigned
integer to store a boolean. There's no real need for this and these
occurrences can be replaced with 'bool'.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>

diff --git a/src/conf/interface_conf.h b/src/conf/interface_conf.h
index 4857822b478b883fc15ff4ad64d87a30ed1a0c0b..0e225750a8a45a95998220773fd03b3e3ba419a8 100644 (file)
--- a/src/conf/interface_conf.h
+++ b/src/conf/interface_conf.h
@@ -164,7 +164,7 @@ typedef virInterfaceObj *virInterfaceObjPtr;
 struct _virInterfaceObj {
     virMutex lock;
 
-    unsigned int active:1;           /* 1 if interface is active (up) */
+    bool active;           /* true if interface is active (up) */
     virInterfaceDefPtr def; /* The interface definition */
 };
 
@@ -175,7 +175,7 @@ struct _virInterfaceObjList {
     virInterfaceObjPtr *objs;
 };
 
-static inline int
+static inline bool
 virInterfaceObjIsActive(const virInterfaceObj *iface)
 {
     return iface->active;

diff --git a/src/rpc/virnetserver.c b/src/rpc/virnetserver.c
index 2306e10e00d470343c6518ff9c87ea1d6b18a95d..89077683f5d3d5cd342b8f75fece9b854dea23d4 100644 (file)
--- a/src/rpc/virnetserver.c
+++ b/src/rpc/virnetserver.c
@@ -95,7 +95,7 @@ struct _virNetServer {
     unsigned int keepaliveCount;
     bool keepaliveRequired;
 
-    unsigned int quit :1;
+    bool quit;
 
 #ifdef WITH_GNUTLS
     virNetTLSContextPtr tls;
@@ -1035,7 +1035,7 @@ static void virNetServerAutoShutdownTimer(int timerid ATTRIBUTE_UNUSED,
 
     if (!srv->autoShutdownInhibitions) {
         VIR_DEBUG("Automatic shutdown triggered");
-        srv->quit = 1;
+        srv->quit = true;
     }
 
     virObjectUnlock(srv);
@@ -1074,7 +1074,7 @@ void virNetServerRun(virNetServerPtr srv)
         virNetServerMDNSStart(srv->mdns) < 0)
         goto cleanup;
 
-    srv->quit = 0;
+    srv->quit = false;
 
     if (srv->autoShutdownTimeout &&
         (timerid = virEventAddTimeout(-1,
@@ -1162,7 +1162,7 @@ void virNetServerQuit(virNetServerPtr srv)
     virObjectLock(srv);
 
     VIR_DEBUG("Quit requested %p", srv);
-    srv->quit = 1;
+    srv->quit = true;
 
     virObjectUnlock(srv);
 }