This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Darwin build fix for unused prototypes
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Josh Matthews <josh at joshmatthews dot net>
- Cc: gdb-patches at sourceware dot org, Tristan Gingold <gingold at adacore dot com>
- Date: Fri, 9 Mar 2012 05:40:14 -0800
- Subject: Re: [patch] Darwin build fix for unused prototypes
- References: <CAOvJiVVT4_Gqw==81sT0bQ5MLfWiZ6ROMqjZRKrsqp2XtZU9_Q@mail.gmail.com>
Hi Josh,
> 2012-03-08 Josh Matthews <josh@joshmatthews.net>
>
> * 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