OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

minor cleanups in is_dynamic_type 

I noticed that gdbtypes.c:is_dynamic_type has some unneeded "break"s.
This patch cleans up the function a bit, removing those and removing
the switch's default case so that the end of the function is a bit
clearer.

2014-06-04  Tom Tromey  <tromey@redhat.com>

	* gdbtypes.c (is_dynamic_type): Remove unneeded "break"s.
This commit is contained in:
Tom Tromey 2014-05-08 10:10:43 -06:00
parent c0939df1ce
commit 92e2a17f9b
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gdb/ChangeLog
View File

@ -1,3 +1,7 @@
2014-06-04 Tom Tromey <tromey@redhat.com>
* gdbtypes.c (is_dynamic_type): Remove unneeded "break"s.
2014-06-04 Tom Tromey <tromey@redhat.com>
* procfs.c (procfs_attach): Make "args" const.

10
gdb/gdbtypes.c
View File

@ -1625,7 +1625,6 @@ is_dynamic_type (struct type *type)
{
case TYPE_CODE_RANGE:
return !has_static_range (TYPE_RANGE_DATA (type));
break;
case TYPE_CODE_ARRAY:
{
@ -1635,14 +1634,11 @@ is_dynamic_type (struct type *type)
or the elements it contains have a dynamic contents. */
if (is_dynamic_type (TYPE_INDEX_TYPE (type)))
return 1;
else
return is_dynamic_type (TYPE_TARGET_TYPE (type));
break;
return is_dynamic_type (TYPE_TARGET_TYPE (type));
}
default:
return 0;
break;
}
return 0;
}
static struct type *