Move defaults for some switches into a default case

This is clearly equivalent, and quiets -Wswitch-default.

Based on part of a patch by Thomas Zimmermann.

Signed-off-by: Simon McVittie <smcv@debian.org>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=98191

diff --git a/bus/config-parser-common.c b/bus/config-parser-common.c
index 5db6b2891f8cf3714dac2d2577c8615bffff1609..c1c4191ba6edee3689d3348abf1c93dcf6519e16 100644 (file)
--- a/bus/config-parser-common.c
+++ b/bus/config-parser-common.c
@@ -187,10 +187,9 @@ bus_config_parser_element_type_to_name (ElementType type)
       return "allow_anonymous";
     case ELEMENT_APPARMOR:
       return "apparmor";
+    default:
+      _dbus_assert_not_reached ("bad element type");
+      return NULL;
     }
-
-  _dbus_assert_not_reached ("bad element type");
-
-  return NULL;
 }
 

diff --git a/dbus/dbus-credentials.c b/dbus/dbus-credentials.c
index 151bb00f99b4bb3ef441c1efbdb71bc9ee112758..5fa754c2cbc37f19d7726c3bc86a304ed24a83a9 100644 (file)
--- a/dbus/dbus-credentials.c
+++ b/dbus/dbus-credentials.c
@@ -267,10 +267,10 @@ _dbus_credentials_include (DBusCredentials    *credentials,
       return credentials->linux_security_label != NULL;
     case DBUS_CREDENTIAL_ADT_AUDIT_DATA_ID:
       return credentials->adt_audit_data != NULL;
+    default:
+      _dbus_assert_not_reached ("Unknown credential enum value");
+      return FALSE;
     }
-
-  _dbus_assert_not_reached ("Unknown credential enum value");
-  return FALSE;
 }
 
 /**

diff --git a/dbus/dbus-memory.c b/dbus/dbus-memory.c
index bba9884ee60dc86b3c6a26c49978e1d48f19c37a..a9c2f7a44da7b76c238a40fc69c392354881a0ac 100644 (file)
--- a/dbus/dbus-memory.c
+++ b/dbus/dbus-memory.c
@@ -334,9 +334,10 @@ source_string (BlockSource source)
       return "malloc0";
     case SOURCE_REALLOC_NULL:
       return "realloc(NULL)";
+    default:
+      _dbus_assert_not_reached ("Invalid malloc block source ID");
+      return "invalid!";
     }
-  _dbus_assert_not_reached ("Invalid malloc block source ID");
-  return "invalid!";
 }
 
 static void

diff --git a/dbus/dbus-sysdeps-win.c b/dbus/dbus-sysdeps-win.c
index 343a51ddfe39e5364cdd5270c465cf9da0d9f2d9..77c5b74e5200ad360a3a320deea2a9616df9a745 100644 (file)
--- a/dbus/dbus-sysdeps-win.c
+++ b/dbus/dbus-sysdeps-win.c
@@ -323,9 +323,10 @@ _dbus_win_error_from_last_error (void)
     case ERROR_FILE_NOT_FOUND:
     case ERROR_PATH_NOT_FOUND:
       return DBUS_ERROR_FILE_NOT_FOUND;
+
+    default:
+      return DBUS_ERROR_FAILED;
     }
-  
-  return DBUS_ERROR_FAILED;
 }
 
 

diff --git a/dbus/dbus-sysdeps.c b/dbus/dbus-sysdeps.c
index d1a96d12fcedf93964678161c4297413e809bf2b..20bb89441969e84ae30a313d329aa778d7f2cd8a 100644 (file)
--- a/dbus/dbus-sysdeps.c
+++ b/dbus/dbus-sysdeps.c
@@ -670,9 +670,9 @@ _dbus_error_from_errno (int error_number)
     case ENOENT:
       return DBUS_ERROR_FILE_NOT_FOUND;
 #endif
+    default:
+      return DBUS_ERROR_FAILED;
     }
-
-  return DBUS_ERROR_FAILED;
 }
 
 /**