Re: [gnugo-devel] eye_data in local_owl_data

[Gunnar Farneback]

Arend wrote:
> In the struct local_owl_data, we have among other things
> 
>   struct eye_data black_eye[BOARDMAX];
>   struct eye_data white_eye[BOARDMAX];
> 
> It seems to me it would be completely sufficient to only use s.th. like
>   struct eye_data my_eye[BOARDMAX];
> which would store the eyes belonging to the color of the dragon being
> attacked.
> 
> Can anyone spot s.th. I might be missing?

I'm not entirely certain that only one would suffice, but I tend to
believe it would. There are some technical complications however. At
least some helper functions like owl_big_eyespace() currently involve
both eyespaces (but they probably shouldn't have to or even wish to).

> This question isn't purely academical, push_owl consumes 4.5% of CPU
> time at the moment, and its time consumption should be pretty much
> proportional to sizeof(local_owl_data).

Yes, I've noticed this too.

/Gunnar