From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7218 invoked by alias); 30 Sep 2002 15:10:21 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 7203 invoked from network); 30 Sep 2002 15:10:19 -0000 Received: from unknown (HELO touchme.toronto.redhat.com) (216.138.202.10) by sources.redhat.com with SMTP; 30 Sep 2002 15:10:19 -0000 Received: from redhat.com (totem.toronto.redhat.com [172.16.14.242]) by touchme.toronto.redhat.com (Postfix) with ESMTP id 53815800195; Mon, 30 Sep 2002 11:10:18 -0400 (EDT) Message-ID: <3D98695A.5020406@redhat.com> Date: Mon, 30 Sep 2002 08:10:00 -0000 From: Fernando Nasser Organization: Red Hat , Inc. - Toronto User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1) Gecko/20020827 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Andrew Cagney Cc: gdb-patches@sources.redhat.com Subject: Re: RFC/RFA: Move new disassembler to libgdb proper References: <3D9394BA.2040605@redhat.com> <3D93BF54.30809@redhat.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2002-09/txt/msg00750.txt.bz2 Andrew Cagney wrote: > 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.''. > OK. I was not sure, as the code is older than that. But I guess the copyright goes by the file, not the code... >> + 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. Yes, how could I forget! > It should take an explicit ui_out parameter. > Good point. Of all people I should have remembered that one ;-) > 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 :-) > It seems that it may need some clean-up. But you are right, I am just moving code and I like to do the least possible changes on that pass. One of us should take a look and see if there is something else. It will be easier to maintain it as it becomes the only disassembler code instead of one of many. Thanks for looking at this. I will make the changes and check it in. Regards, Fernando -- Fernando Nasser Red Hat - Toronto E-Mail: fnasser@redhat.com 2323 Yonge Street, Suite #300 Toronto, Ontario M4P 2C9