URL:
  <http://gna.org/bugs/?21471>

                 Summary: is_req_active can spuriously return FALSE for
RPT_POSSIBLE with continent/city-ranged improvement requirements
                 Project: Freeciv
            Submitted by: jtn
            Submitted on: Sat Jan 11 20:17:08 2014
                Category: None
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: In Progress
             Assigned to: jtn
        Originator Email: 
             Open/Closed: Open
                 Release: 
         Discussion Lock: Any
        Operating System: Any
         Planned Release: 

    _______________________________________________________

Details:

If there's a requirement for an improvement on the same continent or in the
same city, is_req_active() called with RPT_POSSIBLE and no target city can
return FALSE when it should return TRUE.

I don't know if this is breaking anything important in practice, just noticed
it in passing.

Propose to fix by moving tristate logic inside buildings_in_range(), and
completing its half-finished transition from returning a count to a
boolean-ish (no-one uses it in the former sense any more, at least on trunk,
and some code paths return TRUE/FALSE already).




    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?21471>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to