[PATCH c++ 01/12] linux-ppc-low.c: Remove forward declaration, move ppc_arch_setup lower

Simon Marchi simon.marchi@ericsson.com
Tue Nov 3 18:35:00 GMT 2015


On 15-11-03 11:05 AM, Pedro Alves wrote:
> On 11/03/2015 02:26 PM, Simon Marchi wrote:
>> From: Simon Marchi <simon.marchi@polymtl.ca>
>>
>> g++ doesn't like that we forward-declare a variable that is initialized
>> later in the file.  It's easy enough to re-order things to fix it.
> 
> Yeah, doesn't work in C either with -fno-common.
> 
>>
>> Fixes
>>
>> /home/simark/src/binutils-gdb/gdb/gdbserver/linux-ppc-low.c:663:28: error: redefinition of ‘usrregs_info ppc_usrregs_info’
>>  static struct usrregs_info ppc_usrregs_info =
>>                             ^
>> /home/simark/src/binutils-gdb/gdb/gdbserver/linux-ppc-low.c:381:28: note: ‘usrregs_info ppc_usrregs_info’ previously declared here
>>  static struct usrregs_info ppc_usrregs_info;
>>                             ^
>>
>> gdb/gdbserver/ChangeLog:
>>
>> 	* linux-ppc-low.c (ppc_usrregs_info): Remove
>> 	forward-declaration.
>> 	(ppc_arch_setup): Move lower in file.
> 
> OK.
> 
> Thanks,
> Pedro Alves
> 

Pushed all 12 patches, with the typo in the title fixed.

Thanks!



More information about the Gdb-patches mailing list