Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Doug Evans <dje@sebabeach.org>
To: ac131313@redhat.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: [PATCH] update cgen read_memory_func
Date: Wed, 18 Jun 2003 16:08:00 -0000	[thread overview]
Message-ID: <20030618155926.46273B536@mail.sebabeach.org> (raw)
In-Reply-To: <3EF07F22.7040902@redhat.com> (message from Andrew Cagney on Wed, 18 Jun 2003 11:02:58 -0400)

   Date: Wed, 18 Jun 2003 11:02:58 -0400
   From: Andrew Cagney <ac131313@redhat.com>

   > fyi, I'll be checking this in.
   > 
   > 2003-06-17  Doug Evans  <dje@sebabeach.org>
   > 
   > 	* cgen-trace.h (sim_disasm_read_memory): Update args to be compatible
   > 	with disassemble_info:read_memory_func.
   > 	* cgen-trace.c (sim_disasm_read_memory): Ditto.

   Doug.

   Only people with an FSF GDB copyright assignment are permitted to make 
   changes to the FSF's GDB sources.

I'm not sure what you're saying.
Are you saying someone else needs to check this in?
Or are you saying that this patch can't go in at all?

This patch doesn't need a copyright assignment. It's far too small.
[Otherwise I wouldn't have gone to this trouble...]

   > Index: cgen-trace.c
   > ===================================================================
   > RCS file: /cvs/src/src/sim/common/cgen-trace.c,v
   > retrieving revision 1.5
   > diff -u -p -r1.5 cgen-trace.c
   > --- cgen-trace.c	23 Dec 2000 21:52:14 -0000	1.5
   > +++ cgen-trace.c	18 Jun 2003 05:03:40 -0000
   > @@ -322,12 +322,12 @@ sim_disasm_sprintf VPARAMS ((SFILE *f, c
   >  /* Memory read support for an opcodes disassembler.  */
   >  
   >  int
   > -sim_disasm_read_memory (bfd_vma memaddr, bfd_byte *myaddr, int length,
   > +sim_disasm_read_memory (bfd_vma memaddr, bfd_byte *myaddr, unsigned int length,
   >  			struct disassemble_info *info)
   >  {
   >    SIM_CPU *cpu = (SIM_CPU *) info->application_data;
   >    SIM_DESC sd = CPU_STATE (cpu);
   > -  int length_read;
   > +  unsigned length_read;
   >  
   >    length_read = sim_core_read_buffer (sd, cpu, read_map, myaddr, memaddr,
   >  				      length);
   > Index: cgen-trace.h
   > ===================================================================
   > RCS file: /cvs/src/src/sim/common/cgen-trace.h,v
   > retrieving revision 1.2
   > diff -u -p -r1.2 cgen-trace.h
   > --- cgen-trace.h	14 Oct 2002 10:55:39 -0000	1.2
   > +++ cgen-trace.h	18 Jun 2003 05:03:40 -0000
   > @@ -81,7 +81,7 @@ extern int sim_disasm_sprintf (SFILE *, 
   >  #ifdef __BFD_H_SEEN__
   >  struct disassemble_info;
   >  extern int
   > -sim_disasm_read_memory (bfd_vma memaddr_, bfd_byte *myaddr_, int length_,
   > +sim_disasm_read_memory (bfd_vma memaddr_, bfd_byte *myaddr_, unsigned int length_,
   >  			struct disassemble_info *info_);
   >  extern void
   >  sim_disasm_perror_memory (int status_, bfd_vma memaddr_,
   > 




  reply	other threads:[~2003-06-18 16:08 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-18  5:33 Doug Evans
2003-06-18 15:03 ` Andrew Cagney
2003-06-18 16:08   ` Doug Evans [this message]
2003-06-18 16:29     ` Andrew Cagney
2003-06-18 16:51       ` Doug Evans
2003-06-20 17:38       ` Frank Ch. Eigler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030618155926.46273B536@mail.sebabeach.org \
    --to=dje@sebabeach.org \
    --cc=ac131313@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox