[PATCH] Remove struct keyword from section_addr_info

Simon Marchi simark@simark.ca
Sat Mar 24 02:46:00 GMT 2018


On 2018-03-23 04:36 PM, Keith Seitz wrote:
> Buildbot pointed out a failiure in windows-nat.c:
> 
> ../../binutils-gdb/gdb/windows-nat.c:582:10: error: using typedef-name 'section_addr_info' after 'struct'
>    struct section_addr_info *addrs;
>           ^~~~~~~~~~~~~~~~~
> In file included from ../../binutils-gdb/gdb/windows-nat.c:49:0:
> ../../binutils-gdb/gdb/symfile.h:75:37: note: 'section_addr_info' has a previous declaration here
>  typedef std::vector<other_sections> section_addr_info;
>                                      ^~~~~~~~~~~~~~~~~
> 
> A recursive grep of the sources for "struct section_addr_info" reveals one
> additional reference in a comment.  In both cases, this patch simply removes
> the struct keyword.
> 
> gdb/ChangeLog:
> 
> 	* symfile.c (place_section): Remove "struct" from section_addr_info
> 	in comment.
> 	* windows-nat.c (struct safe_symbol_file_add_args) <addrs>: Likewise.

The "Likewise" is not really accurate, since the second entry isn't changing
a comment like the first one.  But otherwise LGTM (I would have considered it
as pretty obvious).

Simon



More information about the Gdb-patches mailing list