Re: [gnugo-devel] resign_3_17.1

[Gunnar Farneback]

Dan wrote:
> It moves the function dragon_weak to utils.c

The dragon_weak() function only consults dragon data. To me it seems
very naturally placed in dragon.c.

Arend wrote:
> I had just put a patch together to fix this differently. IMO, all the
> gtp_...() functions in play_gtp.c should have "const char *s" as
> argument.

What problem would this solve?

I'm reluctant to add const here. I don't see any need to stop
functions from modifying the string or having to fuzz around if they
want to process the string with non-const library functions.

> That's how they are used (except in play_black() and play_white()),
> and gtp.c shouldn't have to worry about the command string possibly
> getting changed on the callback.

gtp.c couldn't care less if the string was changed. It's not worried
at all.

/Gunnar