[Freeciv-Dev] Re: Team research (PR#4213)

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

Per I. Mathisen wrote:
> On Sun, 21 Sep 2003, Jason Short wrote:
> 
>>>Changed. I expanded short_city packet with one field for this. I hope Greg
>>>or Jason can take a look at it to ensure that we don't leak any unwanted
>>>info here; I am not very knowledgable about dumb_city.
>>
>>I don't like it. It muddles the logic of what the city_packet and
>>short_city_packet do. Keeping this logic simple - you either see the
>>city internals or you don't - will be much safer in the long run I think.
> 
> 
> But you don't see city internals,

IMO you should.  For instance you should see the units in the cities.

> and you already send trade info in
> short_city for trade routes...

True.  But here I see no other alternative.

jason