Re: [Scid-users] g++ hates you

[Fulvio <fb...@li...>]

Michael G Thomas wrote:
> src/tkscid.cpp: In function ‘int sc_name_spellcheck(ClientData, 
> Tcl_Interp*, int, const char**)’:
> src/tkscid.cpp:12479:61: warning: array subscript is above array 
> bounds [-Warray-bounds]
>                              corrections[j-1] = corrections[j];
>                                                              ^
> src/tkscid.cpp:12479:61: warning: array subscript is above array 
> bounds [-Warray-bounds]
> src/tkscid.cpp:12479:44: warning: array subscript is above array 
> bounds [-Warray-bounds]
>                              corrections[j-1] = corrections[j];
>
> Noob here but: Should we not be more anal-retentive about these 
> data-modifying functions?
>
>   
Hi,
thanks for reporting.

However, the -Warray-bounds of gcc is not very stable:
http://www.google.com/search?q=array-bounds+gcc+bug

The scid code at line src/tkscid.cpp:12479 seems good to me (maybe a 
little overcomplicated).
Bye,
Fulvio