diff options
| author | Tom Tromey <tromey@redhat.com> | 2014-06-13 13:08:23 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@redhat.com> | 2014-06-20 08:44:19 -0600 |
| commit | 26f2dc3065ea8621b898fe5b98ef19afbf02aaf7 (patch) | |
| tree | 4cd816f400c0d0691cd7f7da3c770cc9320d9662 /gdb/gdb_obstack.h | |
| parent | 698b3e08336ffac76d45d9ddb6a85ae166098140 (diff) | |
make obstack object allocators more type-safe
This changes OBSTACK_ZALLOC and OBSTACK_CALLOC to cast their value to
the correct type. This is more type-safe and also is more in line
with the other object-allocation macros in libiberty.h.
Making this change revealed one trivial error in dwarf2read.c.
On the whole that seems pretty good to me.
Tested by rebuilding.
2014-06-20 Tom Tromey <tromey@redhat.com>
* dwarf2read.c (dw2_get_real_path): Use correct type in
OBSTACK_CALLOC.
* gdb_obstack.h (OBSTACK_ZALLOC, OBSTACK_CALLOC): Cast result.
Diffstat (limited to 'gdb/gdb_obstack.h')
| -rw-r--r-- | gdb/gdb_obstack.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gdb/gdb_obstack.h b/gdb/gdb_obstack.h index 0b394002d2..ec2dd97de9 100644 --- a/gdb/gdb_obstack.h +++ b/gdb/gdb_obstack.h @@ -25,11 +25,11 @@ /* Utility macros - wrap obstack alloc into something more robust. */ #define OBSTACK_ZALLOC(OBSTACK,TYPE) \ - (memset (obstack_alloc ((OBSTACK), sizeof (TYPE)), 0, sizeof (TYPE))) + ((TYPE *) memset (obstack_alloc ((OBSTACK), sizeof (TYPE)), 0, sizeof (TYPE))) #define OBSTACK_CALLOC(OBSTACK,NUMBER,TYPE) \ - (memset (obstack_alloc ((OBSTACK), (NUMBER) * sizeof (TYPE)), \ - 0, (NUMBER) * sizeof (TYPE))) + ((TYPE *) memset (obstack_alloc ((OBSTACK), (NUMBER) * sizeof (TYPE)), \ + 0, (NUMBER) * sizeof (TYPE))) /* Unless explicitly specified, GDB obstacks always use xmalloc() and xfree(). */ |