Re: [gnugo-devel] arend_3_3.3: 1st step reading.c cleanup

[Gunnar Farneback]

Arend wrote:
> I realized that I forgot some more functions that should be called
> .._moves, too. When I started to fix this, I ran into doing some more
> clean-up. Would it be consensus to remove the cache lookups from
> attack1234/defend1234? Caching is disabled for these functions, and I
> don't see any reason why we would want to change this.

I see no reason to keep them. Just make sure that the traces in the
READ_RETURN macros continue to work

> Also, the functions restricted_attack/defend are appartently unused.
> According to the comments, they should be used to produce backfill
> moves, which is done differently now. Should they be removed, or is
> there a reason why they might get used?

They are still potentially useful. Not to generate backfilling moves
in the reading, in my opinion, but they may well be used for similar
purposes in certain helper functions.

/Gunnar