Re: [gnugo-devel] libboard and board.h

[Arend Bayer]

Ok, I have posted one more version of that patch, and I believe this is
the final version. In fact, I think I should merge it quickly to avoid
conflicts in particular with Inge.

Dragon status, safety values, cache routine id's, persistent owl cache
routine id's, komaster state are now all enum's. The only part where
I cheated a little (to not let the size of the patch grow infinitely) is
that all komaster _variables_ are still "int". However, this is
absolutely ok, as the _identifiers_ of an enum are all integer
constants, too.

I think I didn't mention before that I also consistently shrunk the
board library and threw out everything that wasn't directly related to
the board. I.e. the showboard() function in showbord.c knew a lot about
dragon's, and hence showbord is now part of the engine[1]. Only
including board.h means that this will also be automatically enforced
futher on.

Oh, I also moved some definitions from gnugo.h to board.h. It didn't
make sense to keep the board library interface split in 2 parts, and the
distinction between gnugo.h and liberty.h has never quite worked anyway.

Arend

[1] The only reason for it to be in the board library is that showboard(0)
does only board related stuff, but that is also done by simple_showboard(),
which hence got moved to printutils.c.