fix two latent type errors
authorTom Tromey <tromey@redhat.com>
Mon, 19 May 2014 19:10:21 +0000 (13:10 -0600)
committerTom Tromey <tromey@redhat.com>
Mon, 19 May 2014 20:17:16 +0000 (14:17 -0600)
commitc4e54771f84f3acd02c6da80a6f62603eb3e8505
tree26082a646b87994a2b307aeed39d6bf30592bf3f
parent8832e7e38ec3596b4e584cfadca060567a7e8bc3
fix two latent type errors

I'm checking this in as obvious.

I was looking at instances of "alloc.*sizeof" and noticed a couple
where the types in question are incorrect.

In gdbtypes, the code allocates sizeof(int) to represent a struct rank.
In mi-cmds, the code uses "struct mi_cmd **" -- one "*" too many.

In both cases the problems are latent because in practice the sizes
are the same as the sizes of the correct types.  Still, it's better to
be correct.

I think gdb would be improved by a wholesale change from explicit
sizeofs to using the libiberty.h allocation macros.  In most cases
they are both shorter and have better type safety.  However, the
resulting patch is rather large.

Built and regtested on x86-64 Fedora 20.

2014-05-19  Tom Tromey  <tromey@redhat.com>

* gdbtypes.c (rank_function): Use XNEWVEC.
* mi/mi-cmds.c (build_table): Use XCNEWVEC.
gdb/ChangeLog
gdb/gdbtypes.c
gdb/mi/mi-cmds.c