Update of patch #4821 (project freeciv):
Status: None => Ready For Test
Assigned to: None => persia
_______________________________________________________
Follow-up Comment #3:
I was looking forward to this, so I'll rebase and continue. Feel free to take
it back if you want it.
Unfortunately, the logic isn't currently amenable to merging with that in
is_square_threatened(), as in the case of assess_danger, we're also concerned
about being able to take over cities, and in is_square_threatened, we don't
care, causing the booleans to hard to reconcile. This is a useful
optimisation even without flag consolidation and cleanup, and there's enough
other places that we examine threats that it makes sense to have a separate
patch to consolidate that in the future.
The attached patch is mostly pepeto's: I've rebased it over my work from patch
#4831 and added the UTYF_DIPLOMAT check.
(file #21168)
_______________________________________________________
Additional Item Attachment:
File name: pepeto-assess-danger.patch Size:2 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?4821>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
