Re[4]: [gnugo-devel] reading patch

[Arend Bayer]

Paul wrote:

> Arend wrote:
> > I think the problem that the same move gets tried multiple times will
> > go away automatically. Once every function use ADD_CANDIDATE_MOVE, this
> > is automatically checked for.
>
> > There might still be some duplicating work generating these moves,
> > though.
>
> why not use some preventive check: split ADD_CANDIDATE_MOVE to say
> IS_MOVE_IN_LIST, which checks if it is there and the
> ADD_CANDIDATE_MOVE itself, which adds a move without checking. or
> keep a separate version which checks and adds, like ADD_CANDIDATE_MOVE
> does now.

I don't see why we should bother. It's hardly a problem now, and will
go away automatically some day.

Arend