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:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to