Re: [gnugo-devel] Tru64: compilation errors for current CVS

[Arend Bayer]

On Sun, 12 Sep 2004, Teun Burgers wrote:

> Current CVS (12-09-2004) gives errors such as these on .c files
> generated by mkpat:
> 
> cc -DHAVE_CONFIG_H -I. -I. -I.. -I../engine -I../utils -I../sgf
> -I/home/snuf3/aburgers/alpha/include -I/net/shareware/include  -g
> -check_bounds -c `test -f read_attack.c || echo './'`read_attack.c
> cc: Error: read_attack.c, line 136: The static declaration of "read_attack" is
> a tentative definition and specifies an incomplete type. (incompstat)
> static struct pattern read_attack[];
> ----------------------^
> make: *** [read_attack.o] Error 1
> 
> For most patterns files this declaration can simply be left out, but
> not for read_attack.c, because the patterns in read_attack.c refer
> to read_attack before it is actually defined.
> 
> Previously this used to give only warings on Tru64. See this message:
> 
> http://lists.gnu.org/archive/html/gnugo-devel/2004-02/msg00056.html

I had recently changed this. Before it was defined as 'extern', which
was conflicting with the later 'static' definition in the same file. It
gave a compile error with GCC 3.5.

I assume this could be fixed by adding the size of the array into the
forward declaration? I.e. if you add '26' in the line 136:

static struct pattern read_attack[];

Does it compile? (Maybe you also have to add it in line 620.)

Arend