RFA: use constructor to build 'struct regset' objects

Jim Blandy jimb@redhat.com
Thu May 20 03:11:00 GMT 2004


Andrew Cagney <cagney@gnu.org> writes:

> > + struct regset *regset_xmalloc (void *descr,
> > +                                supply_regset_ftype *supply_regset,
> > +                                fill_regset_ftype *fill_regset)
> > + {
> 
> Can you run the file through gdb_indent.sh?

I thought that looked odd.  Thanks; this is fixed in the change I
committed.

> > +   struct regset *r = (struct regset *) xmalloc (sizeof (*r));
> 
> Since this data belongs to an architecture, this should be allocated
> From the architecture's obstack.  I suspect this is a pre-existing
> bug but can you still fix it?

I'll take a look at it.



More information about the Gdb-patches mailing list