gdb-patches Digest 11 Feb 2001 23:49:53 -0000 Issue 545
Andrew Cagney
ac131313@cygnus.com
Fri Feb 16 10:59:00 GMT 2001
Jeffrey A Law wrote:
> > 2001-02-11 Paul Hilfinger <hilfingr@nile.gnat.com>
> >
> > * hpux-thread.c (save_inferior_pid): Change call to make_cleanup
> > to make arguments conform.
> > (restore_inferior_pid): Change signature to that expected by
> > make_cleanup.
> No comment on this one -- I'm not that familiar with the new guidelines
> for make_cleanup. I'll let Andrew comment on this.
It was rejected. It was hiding a type compatibility problem by simply
casting it.
> > 2001-02-11 Paul Hilfinger <hilfingr@lisbon.int.act-europe.fr>
> >
> > * hpux-thread.c (hpux_thread_xfer_memory): Add mem_attrib
> > argument to parameter list and to call in order to conform to
> > to_xfer_memory field of struct target_ops.
> ?!? I can't see how this can be right.
>
> >From target.h in the current CVS tree the to_xfer_memory signature looks
> like this:
>
> int (*to_xfer_memory) PARAMS ((CORE_ADDR memaddr, char *myaddr,
> int len, int write,
> struct target_ops * target));
int (*to_xfer_memory) (CORE_ADDR memaddr, char *myaddr,
int len, int write,
struct mem_attrib *attrib,
struct target_ops *target);
I think it is an obvious fix - I just need to cross check the change
with another target. I was hopeing J.T. would get to it before I did.
Andrew
More information about the Gdb-patches
mailing list