Remove dead code from varobj_set_display_format
authorTom Tromey <tromey@adacore.com>
Thu, 31 Aug 2023 18:11:46 +0000 (12:11 -0600)
committerTom Tromey <tromey@adacore.com>
Thu, 7 Sep 2023 20:04:53 +0000 (14:04 -0600)
varobj_set_display_format takes an enum and exhaustively switches on
the values -- but also has a default.  This default case is dead code.

Reviewed-by: Keith Seitz <keiths@redhat.com>
gdb/varobj.c

index 52e62aca265e10d10bf1d6b74410d5bb516cfd3d..760809c6f5150387e7f8499a35ca63b4a8c9ae7c 100644 (file)
@@ -488,20 +488,7 @@ enum varobj_display_formats
 varobj_set_display_format (struct varobj *var,
                           enum varobj_display_formats format)
 {
-  switch (format)
-    {
-    case FORMAT_NATURAL:
-    case FORMAT_BINARY:
-    case FORMAT_DECIMAL:
-    case FORMAT_HEXADECIMAL:
-    case FORMAT_OCTAL:
-    case FORMAT_ZHEXADECIMAL:
-      var->format = format;
-      break;
-
-    default:
-      var->format = variable_default_display (var);
-    }
+  var->format = format;
 
   if (varobj_value_is_changeable_p (var) 
       && var->value != nullptr && !var->value->lazy ())