From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5358 invoked by alias); 9 Mar 2012 16:35:28 -0000 Received: (qmail 5031 invoked by uid 22791); 9 Mar 2012 16:35:25 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,TW_FN X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 09 Mar 2012 16:35:02 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id D966A1C6866; Fri, 9 Mar 2012 11:35:00 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id UIIsAIfvcm2e; Fri, 9 Mar 2012 11:35:00 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id DE7671C6791; Fri, 9 Mar 2012 11:34:59 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id 2706E145615; Fri, 9 Mar 2012 08:34:51 -0800 (PST) Date: Fri, 09 Mar 2012 16:35:00 -0000 From: Joel Brobecker To: Josh Matthews Cc: gdb-patches@sourceware.org, Tristan Gingold Subject: Re: [patch] Darwin build fix for unused prototypes Message-ID: <20120309163451.GC2853@adacore.com> References: <20120309134014.GB2853@adacore.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-03/txt/msg00338.txt.bz2 Hi Josh > Thanks for the suggestions. Please commit this, as I don't have access. Do you have a copyright assignment on file with FSF. I searched the records, and could not find you... If not, we can acceptthis patch as a "tiny change" (which is also obvious). But if you think that you might make more contributions in the future, it would be better if you had one. Otherwise, the total volume of your contributions would exceed our guidelins for max contribution size without copyright assignment. Thanks, -- Joel > gdb/ChangeLog: > > * darwin-nat-info.c (_initialize_darwin_info_commands): Add > prototype. > (darwin_debug_port_info): Make static. > * darwin-nat.c (_initialize_darwin_inferior): Add prototype. > * machoread.c (_initialize_machoread): Add prototype. > * i386-darwin-nat.c (i386_darwin_dr_set, i386_darwin_dr_get) > (i386_darwin_set_control, i386_darwin_get_control) > i386_darwin_dr_set_addr, i386_darwin_get_addr) > i386_darwin_get_status, i386_darwin_get_control): > Hide these with HW_WATCHPOINT_NOT_YET_ENABLED. > > Cheers, > Josh > > On 9 March 2012 08:40, Joel Brobecker wrote: > > Hi Josh, > > > >> 2012-03-08  Josh Matthews   > >> > >>         * darwin-nat-info.c: Add missing prototype for > >> _initialize_darwin_info_commands. Make darwin_debug_port_info static. > >>         * darwin-nat.c: Add missing prototype for _initialize_darwin_infereior. > >>         * i386-darwin-nat.c: Remove unused functions > >> i386_darwin_dr_set, i386_darwin_dr_get, i386_darwin_set_control, > >> i386_darwin_get_control, i386_darwin_dr_set_addr, > >> i386_darwin_get_addr, i386_darwin_get_status, i386_darwin_get_control. > >>         * machoread.c: Add missing prototype for _initialize_machoread. > > > > Before you commit, can you make sure you re-format this ChangeLog > > entry so that all the text is aligned on a tab? Also, can you use > > two spaces after each period. And finally, I noticed at least one > > typo: > > > >        _initialize_darwin_infereior > > > > I suggest: > > > >        * darwin-nat-info.c (_initialize_darwin_info_commands): Add > >        prototype. > >        (darwin_debug_port_info): Make static. > >        * darwin-nat.c (_initialize_darwin_inferior): Add prototype. > >        * machoread.c (_initialize_machoread): Add prototype. > >        * i386-darwin-nat.c (i386_darwin_dr_set, i386_darwin_dr_get) > >        (i386_darwin_set_control, i386_darwin_get_control) > >        i386_darwin_dr_set_addr, i386_darwin_get_addr) > >        i386_darwin_get_status, i386_darwin_get_control): > >        Comment these functions out. > > > > -- > > Joel > diff --git a/gdb/darwin-nat-info.c b/gdb/darwin-nat-info.c > index 9170e14..9741538 100644 > --- a/gdb/darwin-nat-info.c > +++ b/gdb/darwin-nat-info.c > @@ -301,7 +301,7 @@ info_mach_ports_command (char *args, int from_tty) > } > > > -void > +static void > darwin_debug_port_info (task_t task, mach_port_t port) > { > kern_return_t kret; > @@ -843,6 +843,9 @@ info_mach_exceptions_command (char *args, int from_tty) > } > } > > +/* -Wmissing-prototypes */ > +extern initialize_file_ftype _initialize_darwin_info_commands; > + > void > _initialize_darwin_info_commands (void) > { > diff --git a/gdb/darwin-nat.c b/gdb/darwin-nat.c > index 0feebe3..de661a3 100644 > --- a/gdb/darwin-nat.c > +++ b/gdb/darwin-nat.c > @@ -2015,6 +2015,9 @@ darwin_supports_multi_process (void) > return 1; > } > > +/* -Wmissing-prototypes */ > +extern initialize_file_ftype _initialize_darwin_inferior; > + > void > _initialize_darwin_inferior (void) > { > diff --git a/gdb/i386-darwin-nat.c b/gdb/i386-darwin-nat.c > index b452f1e..ced2903 100644 > --- a/gdb/i386-darwin-nat.c > +++ b/gdb/i386-darwin-nat.c > @@ -260,7 +260,7 @@ i386_darwin_store_inferior_registers (struct target_ops *ops, > } > } > > - > +#ifdef HW_WATCHPOINT_NOT_YET_ENABLED > /* Support for debug registers, boosted mostly from i386-linux-nat.c. */ > > static void > @@ -410,6 +410,7 @@ i386_darwin_dr_get_control (void) > { > return i386_darwin_dr_get (DR_CONTROL); > } > +#endif > > void > darwin_check_osabi (darwin_inferior *inf, thread_t thread) > diff --git a/gdb/machoread.c b/gdb/machoread.c > index 88ce612..8a6b500 100644 > --- a/gdb/machoread.c > +++ b/gdb/machoread.c > @@ -1035,6 +1035,9 @@ static const struct sym_fns macho_sym_fns = { > &psym_functions > }; > > +/* -Wmissing-prototypes */ > +extern initialize_file_ftype _initialize_machoread; > + > void > _initialize_machoread () > { -- Joel