[patch] Darwin build fix for unused prototypes

Josh Matthews josh@joshmatthews.net
Fri Mar 9 15:40:00 GMT 2012


Thanks for the suggestions. Please commit this, as I don't have access.

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 <brobecker@adacore.com> wrote:
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: darwin_prototypes.diff
Type: application/octet-stream
Size: 1772 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20120309/e6022942/attachment.obj>


More information about the Gdb-patches mailing list