Re: [gnugo-devel] improved stabilisation in get_next_move_from_list

[Gunnar Farneback]

Teun wrote:
> This patch replaces the third stabilisation criterion
> in get_next_move_from_list by a orientation independent
> criterion
> 
> The original criterion is based on the move itself:
> 
> && list->pattern_list[bottom].move < top_move))
> 
> The new criterion is based on the distance to the center of the board:
> 
> && bdist(list->pattern_list[bottom].move) < top_dist))

What effects does this have on the performance?

> +static double bdist(int move);
> +static double bdist(int move)
> +{

There's no requirement to have a forward declaration if the function
is not referenced before it's defined and it's pointless to have the
declaration immediately before the definition. In this case I see no
advantage in having a forward declaration at all.

I suppose you didn't really intend the function to return a double.

/Gunnar