[Freeciv-Dev] Re: City styles - patch

[Jerzy Klek <qpkjeek@xxxxxxxxxxxxxxx>]

Hi!

> At first glance, it looks good.
> 
> One small problem:  I tried to load an old save file, and it aborted
> because there was no player0.city_style value.  In player_load() in
> plrhand.c, you should probably use secfile_lookup_int_default(), and, if
> city_style is not found, pick a reasonable default, based on the technology
> level of the player.

Thanks for info, I have fixed this problem. The nation default
style will be used. There's no need to look at player's tech
at this stage. This is check when city is drawn (It's not
optimal, I know).

> Now, I'll look in to implementing the style selection for the Xaw client.

Not much work here, just update nation selection dialog.

/Jerzy

city2_diff.gz
Description: city2_diff.gz