Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-07-10 Thread William Allen Simpson
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 Committed trunk revision 13093. Adopted some of Per's syntax, with my ruleset option, attached for posterity. Index: server/citytools.c === --- server/citytools.c

Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-05-19 Thread Per I. Mathisen
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 On Sat, 19 May 2007, William Allen Simpson wrote: Building civ1 cities didn't reveal unexplored areas! So, my version with the configuration line is better for backward compatibility. I do not see why we should worry about that. It

Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-05-19 Thread William Allen Simpson
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 Per I. Mathisen wrote: I do not see why we should worry about that. It is nice that people can play mods that feel like Civ1/2 with Freeciv, but we do not need to support all kinds of small oddball configurations and/or bugs that

Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-05-17 Thread Per I. Mathisen
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 On Wed, 16 May 2007, William Allen Simpson wrote: One of the nuisances (compared to the commercial games) is the limited vision radius of the city itself. ... When the city is founded, it still has only 1 radius vision. That's

Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-05-17 Thread William Allen Simpson
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 Per I. Mathisen wrote: Yes. Patch to change this most welcome. Apparently, it was a conscious decision by some prior developer. There is a documented vision parameter that is only FALSE for cities. Therefore, I propose a

Re: [Freeciv-Dev] (PR#39370) 2.1.0b4 city vision radius too small

2007-05-17 Thread William Allen Simpson
URL: http://bugs.freeciv.org/Ticket/Display.html?id=39370 Oh bother, I see that while I was poking at keeping things backward compatible, you made a sweeping change. Ah well, here's my 2.1.0b4 proposal as applied to trunk Mix and match?