[Freeciv-Dev] Re: (PR#12490) Watchtower patch

["Per I. Mathisen" <per@xxxxxxxxxxx>]

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

On Sun, 13 Mar 2005, Jason Short wrote:
> >- Moved watchtower options from server options to terrain ruleset
> >- Removed the 'wtowervision' option.
> >- Set default 'wtowerevision' to 2. This increases the vision range of
> > any ground unit in a fortress by 2.
> >- Simplified the code, and increased max 'wtowerevision' from 2 to 4.

> 1.Why terrain.ruleset?  Should be units.ruleset I would think.

I disagree. All the other fortress rules are in terrain.ruleset. There are
no similar rules in units.ruleset.

> 2.A problem I've always had with fortress evision is that fortresses
> are basically the same as city walls, but the evision doesn't apply in
> city walls.Should we change this so that evision DOES apply with city
> walls?(Yeah this isn't directly related, just a side note.)

No. Fortress and city walls are quite different. Fortress is much better.

  - Per