On Tue, 14 Dec 2021 14:56:29 +0100 (CET)
Stefan Fellner wrote:
> you're right - I should only do one thing at a time.
>
> I figured it is problematic - so it needs more thorough testing.
>
> I will try to nail it down one-for-one and describe the (possible)
> improvement i see.
Thanks. This area has been a real nuisance, and once again, really easy
to break.
> > Wrt the second change, I am wondering if we should use hasTile() rather
> > than isInEurope() to include the high seas case.
> >
> So the second change you're right, when the hasTile() includes the HighSeas
> case this is even better, I think that's probably not causing side effects. I
> changed it, and the patch contains only this one change.
I am struggling to think of how this can go wrong, so committed
(git.45c2e4be94). What will be different is that in some cases the
currently selected tile will now no longer change, which might work better
if there is a selectable unit still there. Alas, I have been frequently
wrong in this area so let us hope for some good testing.
Cheers,
Mike Pope
pgp2WpOxi7RTu.pgp
Description: OpenPGP digital signature
___
Freecol-developers mailing list
Freecol-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freecol-developers
