Re: [gnugo-devel] paul_3_17.2a

[Paul Pogonyshev]

Arend wrote:
> Trevor wrote:
> > http://www.public32.com/games/go/vcrepair_3_17.1
> >  - bring VC in sync w/ new -t syntax.
> >
> > Note: a new include file "random.h" snuck into dfa.c.  Breaks under VC,
> > compiles without just fine.  Is it needed?  This patch whacks it.
>
> This is needed for the calls to gg_rand(). You should get warnings
> without it.

exactly. it seems to be the same problem we had when i #include'd
eyes.h into mkeyes.c (or something like that).

> > Also, the Debug build of VC doesn't use the "-t" option.  I don't
> > remember what that used to do and why it's not there.  Should the
> > new option be added there?
>
> I think so.  Otherwise you won't use the optimized DFA rotations in
> debug builds. This can lead to unexpected owl results, probably not
> something you want.

you don't remember because it's just been added. that's why all the
problems with vc have arisen.

Paul