[Freeciv-Dev] Re: (PR#12950) Failed sanity check in latest cvs.

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=12950 >

Benedict Adamson wrote:

> I'd say that was crying out for a function called add_unit_sight_points. 
> I did a bit more work on the problem between my previous post and yours, 
> producing the following:

Problem is you usually have to add on the new sight points before
removing the existing sight points, even if this seems backwards.

When adding a fortress:

  add sight points from watchtower
  remove sight points from no watchtower

When removing a fortress:

  add sight points from no-watchtower
  remove sign points from watchtower

which is why there is no such helper code yet (but there is a ticket for
it somewhere, stalled).

-jason