Freeciv

The variable out is assigned twice, shouldn't there be TRI_AND() whenever we get TRI_MAYBE in a later check? This is mostly harmless since clients make few difference between TRI_YES and TRI_MAYBE and servers should always get definite results. Ah, actres_possible() has a builtin effective tri-and with its def parameter. Thus we should just correct the distance check in main since player actions won't have an actor tile whatever the ATK_* is.