[Freeciv-Dev] Re: (PR#2507) incite cost patch

["Mike Kaufman via RT" <rt@xxxxxxxxxxxxxx>]

On Sun, Dec 08, 2002 at 03:44:45AM -0800, Per I. Mathisen via RT wrote:
> 
> On Sat, 7 Dec 2002, Mike Kaufman via RT wrote:
> > > +/* Distance from capital */
> > > +capital = find_palace(city_owner(pcity));
> > > +if (capital) {
> > > +  int tmp = map_distance(capital->x, capital->y, pcity->x, pcity->y);
> >
> > considering that a modpack could have multiple capital cities, and I
> > suppose you want the minimum distance, this could get ugly...
> 
> Multiple capitals is not supported by any code as of yet. I see no reason
> to start here.

real-soon-now (TM) you will easily be able to define multiple Capital_City
effects. What would you do in this case? I'm unsure that find_palace() will
be able to take care of it...

> 
> > > +if (city_got_building(pcity, B_COURTHOUSE)) {
> > > +  dist /= 4;
> > > +}
> >
> > what? the ruleset help mentions /=2 (as well as the current code) changing
> > the rules?
> 
> Yes. I should update the ruleset, too.

please.

> 
> Or did you find the proposed change objectionable?

others may. I have no opinion.

-mike