RFC/RFA: Move new disassembler to libgdb proper

Andrew Cagney ac131313@redhat.com
Thu Sep 26 19:15:00 GMT 2002


Two tweeks:

> + /* Disassemble support for GDB.
> +    Copyright 2000, 2001, 2002 Free Software Foundation, Inc.
> +    Contributed by Cygnus Solutions (a Red Hat company).

The (C) should just be 2002 and ``Red Hat, Inc.''.

> +    This file is part of GDB.
> + 
> +    This program is free software; you can redistribute it and/or modify
> +    it under the terms of the GNU General Public License as published by
> +    the Free Software Foundation; either version 2 of the License, or
> +    (at your option) any later version.
> + 
> +    This program is distributed in the hope that it will be useful,
> +    but WITHOUT ANY WARRANTY; without even the implied warranty of
> +    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +    GNU General Public License for more details.
> + 
> +    You should have received a copy of the GNU General Public License
> +    along with this program; if not, write to the Free Software
> +    Foundation, Inc., 59 Temple Place - Suite 330,
> +    Boston, MA 02111-1307, USA.  */
> + 
> + extern void gdb_disassembly (char *file_string,
> + 			     int line_num,
> + 			     int mixed_source_and_assembly,
> + 			     int how_many, CORE_ADDR low, CORE_ADDR high);

It needs #ifdef DISASM_H et.al.
It should take an explicit ui_out parameter.

As for the rest vis:

> ! static int
> ! gdb_dis_asm_read_memory (bfd_vma memaddr, bfd_byte * myaddr,
> ! 			 unsigned int len, disassemble_info * info)
> ! {
> !   extern struct target_ops exec_ops;

I guess we all get to grit our teeth because we know you're just moving 
around existing code :-)

Andrew




More information about the Gdb-patches mailing list