Makefile: WARNINGS: add -Wswitch-default
authorFerry Huberts <ferry.huberts@pelagic.nl>
Tue, 29 May 2012 12:23:13 +0000 (14:23 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Tue, 29 May 2012 13:19:31 +0000 (15:19 +0200)
-Wswitch-default
  Warn whenever a "switch" statement does not have a "default" case.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Makefile.inc
lib/nameservice/src/nameservice.c
lib/p2pd/src/p2pd.c
lib/secure/src/olsrd_secure.c
lib/tas/Makefile
lib/tas/src/http.c
src/build_msg.c
src/cfgparser/local.mk
src/hashing.c

index 541e9fb..4d4e35e 100644 (file)
@@ -93,6 +93,7 @@ WARNINGS +=   -Wdouble-promotion
 WARNINGS +=    -Wformat-security
 WARNINGS +=    -Wformat-y2k
 WARNINGS +=    -Winit-self
+WARNINGS +=    -Wswitch-default
 # the following 2 do not work yet and need more work on it
 #WARNINGS +=   -Wconversion
 #WARNINGS +=   -Wredundant-decls
index cc09646..53ad64b 100644 (file)
@@ -448,6 +448,9 @@ remove_nonvalid_names_from_list(struct name_entry *my_list, int type)
   case NAME_LATLON:
     valid = is_latlon_wellformed(my_list->name);
     break;
+  default:
+       valid = false;
+       break;
   }
 
   if (!valid) {
index 7fc6316..5263520 100644 (file)
@@ -906,12 +906,6 @@ AddUdpDestPort(const char *value,
   // IPv4 broadcast or IPv6 multicast.
 
   switch (ip_version) {
-  case AF_INET:
-    res = inet_pton(AF_INET, destAddr, &addr4.sin_addr);
-    if (!is_broadcast(addr4) && !is_multicast(addr4)) {
-      OLSR_PRINTF(1,"WARNING: IPv4 address must be multicast or broadcast... ");
-    }
-    break;
   case AF_INET6:
     res = inet_pton(AF_INET6, destAddr, &addr6.sin6_addr);
     if (addr6.sin6_addr.s6_addr[0] != 0xFF) {
@@ -919,6 +913,13 @@ AddUdpDestPort(const char *value,
       return -1;
     }
     break;
+  case AF_INET:
+  default:
+    res = inet_pton(AF_INET, destAddr, &addr4.sin_addr);
+    if (!is_broadcast(addr4) && !is_multicast(addr4)) {
+      OLSR_PRINTF(1,"WARNING: IPv4 address must be multicast or broadcast... ");
+    }
+    break;
   }
   // Determine whether it is a valid IP address
   if (res == 0) {
@@ -935,14 +936,15 @@ AddUdpDestPort(const char *value,
 
   new->ip_version = ip_version;
   switch (ip_version) {
-  case AF_INET:
-    new->address.v4.s_addr = addr4.sin_addr.s_addr;
-    break;
   case AF_INET6:
     memcpy(&new->address.v6.s6_addr,
            &addr6.sin6_addr.s6_addr,
            sizeof(addr6.sin6_addr.s6_addr));
     break;
+  default:
+  case AF_INET:
+    new->address.v4.s_addr = addr4.sin_addr.s_addr;
+    break;
   }
   new->port = destPort;
   new->next = UdpDestPortList;
index b83dc18..a2a046c 100644 (file)
@@ -431,7 +431,8 @@ validate_packet(struct interface *olsr_if, const char *pck, int *size)
     case (SCHEME):
       goto one_checksum_SHA;    /* Ahhh... fix this */
       break;
-
+    default:
+       break;
     }
     break;
 
index abdff3e..c164dcf 100644 (file)
@@ -65,6 +65,7 @@ $(PLUGIN_FULLNAME): $(OBJS) version-script.txt
 
 # this is not nice but there is no nicer solution
 src/lua/lapi.o src/lua/ltable.o:  CFLAGS += -Wno-cast-qual
+src/lua/ldebug.o src/lua/ldo.o src/lua/lbaselib.o src/lua/ldblib.o src/lua/lvm.o src/lua/lapi.o: CFLAGS += -Wno-switch-default
 
 install:       $(PLUGIN_FULLNAME)
                $(STRIP) $(PLUGIN_FULLNAME)
index 01a1a23..183c87a 100644 (file)
@@ -1653,6 +1653,8 @@ serviceConn(struct connInfo *info)
       return -1;
 
     return 0;
+  default:
+         return -1;
   }
 
   return 0;
index ebca6b4..b10830d 100644 (file)
@@ -135,10 +135,11 @@ queue_hello(struct hello_message * message, struct interface * ifp)
 #endif
 
   switch (olsr_cnf->ip_version) {
-  case (AF_INET):
-    return serialize_hello4(message, ifp);
   case (AF_INET6):
     return serialize_hello6(message, ifp);
+  case (AF_INET):
+  default:
+    return serialize_hello4(message, ifp);
   }
   return false;
 }
@@ -164,10 +165,11 @@ queue_tc(struct tc_message * message, struct interface * ifp)
 #endif
 
   switch (olsr_cnf->ip_version) {
-  case (AF_INET):
-    return serialize_tc4(message, ifp);
   case (AF_INET6):
     return serialize_tc6(message, ifp);
+  case (AF_INET):
+  default:
+    return serialize_tc4(message, ifp);
   }
   return false;
 }
@@ -188,10 +190,11 @@ queue_mid(struct interface * ifp)
 #endif
 
   switch (olsr_cnf->ip_version) {
-  case (AF_INET):
-    return serialize_mid4(ifp);
   case (AF_INET6):
     return serialize_mid6(ifp);
+  case (AF_INET):
+  default:
+    return serialize_mid4(ifp);
   }
   return false;
 }
@@ -211,10 +214,11 @@ queue_hna(struct interface * ifp)
 #endif
 
   switch (olsr_cnf->ip_version) {
-  case (AF_INET):
-    return serialize_hna4(ifp);
   case (AF_INET6):
     return serialize_hna6(ifp);
+  case (AF_INET):
+  default:
+    return serialize_hna4(ifp);
   }
   return false;
 }
index b614a9f..0a3212a 100644 (file)
@@ -60,7 +60,7 @@ $(C)oscan.c: $(C)oscan.lex $(C)Makefile
 
 # we need a dependency to generate oparse before we compile oscan.c
 $(C)oscan.o: $(C)oparse.c
-$(C)oscan.o: CFLAGS := $(filter-out -Wunreachable-code -Wsign-compare,$(CFLAGS)) -Wno-sign-compare
+$(C)oscan.o: CFLAGS := $(filter-out -Wunreachable-code -Wsign-compare,$(CFLAGS)) -Wno-sign-compare -Wno-switch-default
 # we need potentially another -I directory
 $(C)oscan.o: CPPFLAGS += $(if $(CFGDIR),-I$(CFGDIR)) -DYY_NO_INPUT
 
index 28a0584..a283c72 100644 (file)
@@ -125,7 +125,10 @@ jenkins_hash(const uint8_t * k, uint32_t length)
     /* no break */
   case 1:
     a += k[0];
-    /* no break */
+    break;
+
+  default:
+         break;
   }
   __jhash_mix(a, b, c);