Re: [Freeciv-Dev] (PR#40034) [Patch] Fix SDL -client compilation

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40034 >

Marko Lindqvist wrote:
>  Will commit immediately.
> 
Great!  Anybody that can do the same for gui-win32?



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] (PR#40034) [Patch] Fix SDL -client compilation

2008-01-21 Thread Marko Lindqvist

http://bugs.freeciv.org/Ticket/Display.html?id=40034 >

 TRUNK and S2_2:
  - Include game.h in a number of c -files that require it

 S2_1:
 - Replace reference to non-existent variable with correct one


 Will commit immediately.


 - ML

diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/cma_fe.c freeciv/client/gui-sdl/cma_fe.c
--- freeciv/client/gui-sdl/cma_fe.c	2008-01-15 04:54:53.0 +0200
+++ freeciv/client/gui-sdl/cma_fe.c	2008-01-22 03:30:14.0 +0200
@@ -20,6 +20,9 @@
 /* utility */
 #include "fcintl.h"
 
+/* common */
+#include "game.h"
+
 /* client */
 #include "civclient.h" /* can_client_issue_orders() */
 
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/dialogs.c freeciv/client/gui-sdl/dialogs.c
--- freeciv/client/gui-sdl/dialogs.c	2008-01-15 04:54:53.0 +0200
+++ freeciv/client/gui-sdl/dialogs.c	2008-01-22 03:31:09.0 +0200
@@ -32,6 +32,7 @@
 
 /* common */
 #include "combat.h"
+#include "game.h"
 #include "government.h"
 #include "movement.h"
 #include "unitlist.h"
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/gui_main.c freeciv/client/gui-sdl/gui_main.c
--- freeciv/client/gui-sdl/gui_main.c	2008-01-15 04:54:53.0 +0200
+++ freeciv/client/gui-sdl/gui_main.c	2008-01-22 03:31:39.0 +0200
@@ -46,6 +46,7 @@
 #include "netintf.h"
 
 /* common */
+#include "game.h"
 #include "unitlist.h"
 
 /* client */
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/mapctrl.c freeciv/client/gui-sdl/mapctrl.c
--- freeciv/client/gui-sdl/mapctrl.c	2008-01-15 04:54:53.0 +0200
+++ freeciv/client/gui-sdl/mapctrl.c	2008-01-22 03:32:07.0 +0200
@@ -29,6 +29,7 @@
 #include "log.h"
 
 /* common */
+#include "game.h"
 #include "unit.h"
 #include "unitlist.h"
 
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/menu.c freeciv/client/gui-sdl/menu.c
--- freeciv/client/gui-sdl/menu.c	2008-01-15 04:54:53.0 +0200
+++ freeciv/client/gui-sdl/menu.c	2008-01-22 03:34:43.0 +0200
@@ -30,6 +30,7 @@
 #include "log.h"
 
 /* common */
+#include "game.h"
 #include "unitlist.h"
 
 /* client */
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/plrdlg.c freeciv/client/gui-sdl/plrdlg.c
--- freeciv/client/gui-sdl/plrdlg.c	2008-01-15 04:54:54.0 +0200
+++ freeciv/client/gui-sdl/plrdlg.c	2008-01-22 03:35:10.0 +0200
@@ -21,6 +21,9 @@
 #include "astring.h"
 #include "fcintl.h"
 
+/* common */
+#include "game.h"
+
 /* client */
 #include "civclient.h"
 #include "climisc.h"
diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/menu.c freeciv/client/gui-sdl/menu.c
--- freeciv/client/gui-sdl/menu.c	2008-01-15 04:04:33.0 +0200
+++ freeciv/client/gui-sdl/menu.c	2008-01-22 03:58:21.0 +0200
@@ -1064,7 +1064,7 @@
   _("Form Traderoute with %s ( %d R&G + %d trade ) (R)"),
   city_name(pHomecity),
   revenue,
-  trade_between_cities(pHomecity, pCity));
+  trade_between_cities(pHomecity, pTile->city));
 } else {
   revenue = (revenue + 2) / 3;
   my_snprintf(cBuf, sizeof(cBuf),
___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40033) sanity checking and debugging improvements

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40033 >

Committed S2_1 revision 14267.
Committed S2_2 revision 14268.
Committed trunk revision 14269.



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] (PR#40033) sanity checking and debugging improvements

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40033 >

Some helpful debugging improvements, and other niceties found along the way:

* sanity checking with file and line

* clear city and unit pointers on free()

* always pair create_unit_virtual() with destroy_unit_virtual()

* moved kill_dying_players() out of player packet handling (plrhand.c)
into server main (srv_main.c), the only place it's called now (3 times).

* for MacOSX, die() doesn't give a traceback.  Fake a bad memory access.
Only in nation_of_player().  Probably should do this elsewhere, too.

Index: server/srv_main.c
===
--- server/srv_main.c   (revision 14266)
+++ server/srv_main.c   (working copy)
@@ -557,6 +557,33 @@
   } players_iterate_end;
 }
 
+/
+  Check all players to see whether they are dying.
+
+  WARNING: do not call this while doing any handling of players, units,
+  etc.  If a player dies, all his units will be wiped and other data will
+  be overwritten.
+
+  FIXME: merge is_alive (105) with is_dying (8) and surrendered (7)?
+/
+static void kill_dying_players(void)
+{
+  players_iterate(pplayer) {
+if (pplayer->is_alive) {
+  /* cities or units remain? */
+  if (0 == city_list_size(pplayer->cities)
+   && 0 == unit_list_size(pplayer->units)) {
+   pplayer->is_dying = TRUE;
+  }
+  /* also F_GAMELOSS in unittools server_remove_unit() */
+  if (pplayer->is_dying) {
+   pplayer->is_dying = FALSE; /* Can't get more dead than this. */
+   kill_player(pplayer);
+  }
+}
+  } players_iterate_end;
+}
+
 /**
   Called at the start of each (new) phase to do AI activities.
 **/
Index: server/settlers.c
===
--- server/settlers.c   (revision 14266)
+++ server/settlers.c   (working copy)
@@ -1343,7 +1343,7 @@
  -result.result : result.result);
 pcity->ai.founder_boat = result.overseas;
   }
-  free(virtualunit);
+  destroy_unit_virtual(virtualunit);
 }
 
 /**
@@ -1377,7 +1377,7 @@
   NULL);
   want = (want - unit_food_upkeep(virtualunit) * FOOD_WEIGHTING) * 100
  / (40 + unit_foodbox_cost(virtualunit));
-  free(virtualunit);
+  destroy_unit_virtual(virtualunit);
 
   /* Massage our desire based on available statistics to prevent
* overflooding with worker type units if they come cheap in
Index: server/sanitycheck.c
===
--- server/sanitycheck.c(revision 14266)
+++ server/sanitycheck.c(working copy)
@@ -25,41 +25,47 @@
 #include "map.h"
 #include "movement.h"
 #include "player.h"
+#include "specialist.h"
 #include "terrain.h"
 #include "unit.h"
 #include "unitlist.h"
 
 #include "citytools.h"
 #include "maphand.h"
-#include "sanitycheck.h"
+#include "srv_main.h"
 #include "unittools.h"
 
+#include "sanitycheck.h"
+
 #ifdef SANITY_CHECKING
 
 #define SANITY_CHECK(x)
\
   do { \
 if (!(x)) {
\
-  freelog(LOG_ERROR, "Failed sanity check: %s (%s:%d)",\
- #x, __FILE__,__LINE__);   \
+  freelog(LOG_ERROR, "Failed %s:%d (%s:%d): %s",   \
+ __FILE__,__LINE__, file, line, #x);   \
 }  \
   } while(0)
 
 #define SANITY_TILE(ptile, check)  \
   do { \
 if (!(check)) {\
-  freelog(LOG_ERROR, "Failed sanity check at %s (%d, %d): "
\
-  "%s (%s:%d)", ptile->city ? city_name(ptile->city)   \
-  : ptile->terrain->name.vernacular, TILE_XY(ptile),   \
-  #check, __FILE__,__LINE__);  \
+  struct city *pcity = tile_get_city(ptile);   \
+  freelog(LOG_ERROR, "Failed %s:%d (%s:%d) at %s (%d,%d): %s", \
+ __FILE__,__LINE__, file, line,\
+ NULL != pcity ? city_name(pcity)  \
+ : terrain_rule_name(tile_get_terrain(ptile)), \
+ TILE_XY(ptile), #check);  \
 }

Re: [Freeciv-Dev] i18n: FR: several nation_adj

2008-01-21 Thread Daniel Markstedt
On 1/22/08, Egor Vyscrebentsov <[EMAIL PROTECTED]> wrote:
> Good daytime!
>
> In some languages (f.e. russian) there are 4 genders for adjectives.
> For example:
> Polish territory - польская (polskaya) территория;
> Polish diplomat - польский (polskiy) дипломат;
> polish unit - польское (polskoye) соединение;
> Polish musketeers - польские (polskiye) мушкетёры.
>
> Is there any possibility to allow to use all of them?
> (I think that this require new X_() macro and convention about number
> of genders in *.nation. I'm afraid this problem couldn't be solved
> by gettext now...)
>
> PS. A long time ago i tried to wrote a proof of solution that used
> gettext plurals. It was very ugly and ineffective...
>
> PPS. I don't speak about cases, those case is really a monster.
>

Swedish would need three forms of adjectives: neuter, common, and plural.

Following Egor's examples:
Polish territory - polskt territorium (neuter)
Polish diplomat - polsk diplomat (common)
Polish musketeers - polska musketörer (plural)

There are some very ugly workarounds in place right now, including
loads of slashes with all the potentially required suffix letters.

Best,
 ~Daniel
___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] gui-sdl lines for translation

2008-01-21 Thread Egor Vyscrebentsov
[Just talk] Shouldn't this message be pluralized? ;-)

#: client/gui-sdl/wldlg.c:1481
#, c-format
msgid ""
"(%d/%d/%d)\n"
"%d/%d %s\n"
"%d %s"
msgstr ""

Seriously, it's hard to understand aout what this message is without
looking into the code. And it seems to me that msgstr will be equal to
msgid here.

-- 
Thanks, evyscr

___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#39983) BUG: gui-xaw shows bad tiles

2008-01-21 Thread Egor Vyscrebentsov

http://bugs.freeciv.org/Ticket/Display.html?id=39983 >

On Fri, 28 Dec 2007 Egor Vyscrebentsov wrote:
> On Sun, 23 Dec 2007 William Allen Simpson wrote:
> > Also, note the little window above the worker at the left, the text is
> > cut off, and doesn't scroll.  When we divide text lines (PR#8737) to
> > shorten them for translation, it will be much worse!
>
> Will be fixed ASAP.

This problem appears only when compiled with debug. Reason is that there
is extra line added in text.c(?) function (was mentioned by me years ago.)
I didn't find proper solution for debug mode which will not break
regular mode. (I don't remember what was when extra line was added in
regular mode, however.)

Problem line in data/Freeciv.in is (2625):

Freeciv*unitinfolabel.label: \n\n\n\n\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ 
\ 

-- 
Thanks, evyscr



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] i18n: FR: several nation_adj

2008-01-21 Thread Egor Vyscrebentsov
Good daytime!

In some languages (f.e. russian) there are 4 genders for adjectives.
For example:
Polish territory - польская (polskaya) территория;
Polish diplomat - польский (polskiy) дипломат;
polish unit - польское (polskoye) соединение;
Polish musketeers - польские (polskiye) мушкетёры.

Is there any possibility to allow to use all of them?
(I think that this require new X_() macro and convention about number
of genders in *.nation. I'm afraid this problem couldn't be solved
by gettext now...)

PS. A long time ago i tried to wrote a proof of solution that used
gettext plurals. It was very ugly and ineffective...

PPS. I don't speak about cases, those case is really a monster.

-- 
Thanks, evyscr

___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] (PR#40032) 2_1:server/plrhand.c:1730 should be plural

2008-01-21 Thread Egor Vyscrebentsov

http://bugs.freeciv.org/Ticket/Display.html?id=40032 >

Good daytime!

branches/2_1

server/plrhand.c:1730 should be plural:
...
"  and the upstart %s now holds power in %d rebel provinces."

PS. [offtopic] There are too much messages which need cases for %s...

-- 
Thanks, evyscr



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] Freeciv - Covermount authorization for Hype!, a portuguese videogames magazine

2008-01-21 Thread Jorge Vieira
OK. Thanks for the feedback on this subject. We'll distribute the source
code along with program, as mentioned in the GNU GPL license requirements.

Best regards,
Jorge

-Original Message-
From: William Allen Simpson [mailto:[EMAIL PROTECTED] 
Sent: segunda-feira, 21 de Janeiro de 2008 17:11
To: Jorge Vieira
Cc: freeciv-dev@gna.org; [EMAIL PROTECTED]
Subject: Re: [Freeciv-Dev] Freeciv - Covermount authorization for Hype!, a
portuguese videogames magazine

Egor Vyscrebentsov wrote:
> Good day!
> 
> On Wed, 16 Jan 2008 Jorge Vieira wrote:
> 
>> Cheers.
>>
>> My name is Jorge Vieira and i'm editor in chief of Hype!, a Portuguese
>> videogames magazine.
>>
>> The magazine has an print run of 30.000 copies, 22.000 of them including
1
>> covermount DVD with full games, demos, software and other materials
related
>> to the videogames universe.
>>
>> We would like to know if you give us permission to include Freeciv in our
>> DVD. We think our readers would appreciate this action and you could also
>> benefit from the attention.
> 
> There was a discussion about sources on the same disk and the result was
> fixed in http://freeciv.wikia.com/wiki/License
> 
> You also may contact [EMAIL PROTECTED] (but I think it will give
> the same result)
> 
> Hope this helps.
> 
And you might wait a few days until the upcoming 2.1.3 release.


-- 
No virus found in this incoming message.
Checked by AVG Free Edition. 
Version: 7.5.516 / Virus Database: 269.19.7/1234 - Release Date: 20-01-2008
14:15



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] Freeciv - Covermount authorization for Hype!, a portuguese videogames magazine

2008-01-21 Thread William Allen Simpson
Egor Vyscrebentsov wrote:
> Good day!
> 
> On Wed, 16 Jan 2008 Jorge Vieira wrote:
> 
>> Cheers.
>>
>> My name is Jorge Vieira and i'm editor in chief of Hype!, a Portuguese
>> videogames magazine.
>>
>> The magazine has an print run of 30.000 copies, 22.000 of them including 1
>> covermount DVD with full games, demos, software and other materials related
>> to the videogames universe.
>>
>> We would like to know if you give us permission to include Freeciv in our
>> DVD. We think our readers would appreciate this action and you could also
>> benefit from the attention.
> 
> There was a discussion about sources on the same disk and the result was
> fixed in http://freeciv.wikia.com/wiki/License
> 
> You also may contact [EMAIL PROTECTED] (but I think it will give
> the same result)
> 
> Hope this helps.
> 
And you might wait a few days until the upcoming 2.1.3 release.

___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40011) client crash with bad tile(s)

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40011 >

Committed S2_1 revision 14262.
Committed S2_2 revision 14263.
Committed trunk revision 14264.



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] Freeciv - Covermount authorization for Hype!, a portuguese videogames magazine

2008-01-21 Thread Egor Vyscrebentsov
Good day!

On Wed, 16 Jan 2008 Jorge Vieira wrote:

> Cheers.
>
> My name is Jorge Vieira and i'm editor in chief of Hype!, a Portuguese
> videogames magazine.
>
> The magazine has an print run of 30.000 copies, 22.000 of them including 1
> covermount DVD with full games, demos, software and other materials related
> to the videogames universe.
>
> We would like to know if you give us permission to include Freeciv in our
> DVD. We think our readers would appreciate this action and you could also
> benefit from the attention.

There was a discussion about sources on the same disk and the result was
fixed in http://freeciv.wikia.com/wiki/License

You also may contact [EMAIL PROTECTED] (but I think it will give
the same result)

Hope this helps.

-- 
Egor Vyscrebentsov,
  as one of Freeciv Development team

___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40011) client crash with bad tile(s)

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40011 >

OK, these two savegames crash in several different ways, but I've got both
running (not crashing) now.  Even though the server tells us the tile is not
"unknown", it still isn't sending valid terrain or sometimes special data.

For those that want to hand repair their files, the error logging indicates
location of the problem.  Of course, we'll probably get a load of complaints,
but that will help indicate the extent and frequency of this problem.

It still adds a city on top of the bad spot, which looks a little silly, but
at least it doesn't crash  Otherwise, I didn't bother to check for the
underlying problem -- any error detected, it leaves the terrain blank/black.

This does not fix any server issues.  That will have to wait.

Index: client/tilespec.c
===
--- client/tilespec.c   (revision 14261)
+++ client/tilespec.c   (working copy)
@@ -2913,35 +2913,35 @@
   Assemble some data that is used in building the tile sprite arrays.
 (map_x, map_y) : the (normalized) map position
   The values we fill in:
-ttype  : the terrain type of the tile
-tspecial   : all specials the tile has
-ttype_near : terrain types of all adjacent terrain
+tterrain_near  : terrain types of all adjacent terrain
 tspecial_near  : specials of all adjacent terrain
 **/
 static void build_tile_data(const struct tile *ptile,
-   struct terrain **tterrain,
-   bv_special *tspecial,
+   struct terrain *pterrain,
struct terrain **tterrain_near,
bv_special *tspecial_near)
 {
   enum direction8 dir;
 
-  *tspecial = tile_get_special(ptile);
-  *tterrain = tile_get_terrain(ptile);
-
   /* Loop over all adjacent tiles.  We should have an iterator for this. */
   for (dir = 0; dir < 8; dir++) {
 struct tile *tile1 = mapstep(ptile, dir);
 
 if (tile1 && client_tile_get_known(tile1) != TILE_UNKNOWN) {
-  tterrain_near[dir] = tile_get_terrain(tile1);
-  tspecial_near[dir] = tile_get_special(tile1);
-} else {
-  /* We draw the edges of the (known) map as if the same terrain just
-   * continued off the edge of the map. */
-  tterrain_near[dir] = *tterrain;
-  BV_CLR_ALL(tspecial_near[dir]);
+  struct terrain *terrain1 = tile_get_terrain(tile1);
+
+  if (NULL != terrain1) {
+tterrain_near[dir] = terrain1;
+tspecial_near[dir] = tile_get_special(tile1);
+continue;
+  }
+  freelog(LOG_ERROR, "build_tile_data() tile (%d,%d) has no terrain!",
+  TILE_XY(tile1));
 }
+/* At the edges of the (known) map, pretend the same terrain continued
+ * past the edge of the map. */
+tterrain_near[dir] = pterrain;
+BV_CLR_ALL(tspecial_near[dir]);
   }
 }
 
@@ -3477,14 +3477,15 @@
   Fill in the sprite array for blended terrain.
 /
 static int fill_terrain_sprite_blending(const struct tileset *t,
- struct drawn_sprite *sprs,
- const struct tile *ptile,
- struct terrain **tterrain_near)
+   struct drawn_sprite *sprs,
+   const struct tile *ptile,
+   const struct terrain *pterrain,
+   struct terrain **tterrain_near,
+   struct terrain_drawing_data *draw)
 {
   struct drawn_sprite *saved_sprs = sprs;
-  struct terrain *pterrain = tile_get_terrain(ptile);
 
-  if (t->is_isometric && t->sprites.terrain[pterrain->index]->is_blended) {
+  if (t->is_isometric && draw->is_blended) {
 enum direction4 dir;
 const int W = t->normal_tile_width, H = t->normal_tile_height;
 const int offsets[4][2] = {
@@ -3574,11 +3575,11 @@
 struct drawn_sprite *sprs,
 int l, /* layer_num */
 const struct tile *ptile,
-struct terrain **tterrain_near)
+const struct terrain *pterrain,
+struct terrain **tterrain_near,
+struct terrain_drawing_data *draw)
 {
   struct drawn_sprite *saved_sprs = sprs;
-  struct terrain *pterrain = tile_get_terrain(ptile);
-  struct terrain_drawing_data *draw = t->sprites.terrain[pterrain->index];
   int match_type = draw->layer[l].match_type;
   int ox = draw->layer[l].offset_x;
   int oy = draw->layer[l].offset_y;
@@ -3773,11 +3774,11 @@
 struct drawn_s

[Freeciv-Dev] (PR#40011) client crash with bad borders, city, and tile owners

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40011 >

Merged with previous report.

I'm thinking that my initial approach to the solution is wrong.  While 
better server checking in 2.2 should catch many problems, the clients 
SHOULD NOT be this fragile.  No matter how badly the server behaves, 
the clients should not crash merely trying to display the bad result.

In this combined ticket, I'll try to improve the 2.1 client robustness.


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40030) Segfault while drawing map

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40030 >

2.2 has more error checking, and here's what it found:

1: Failed sanitycheck.c:231 (cityturn.c:215) in Bellingshausen[2](58,83): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Bellingshausen[2](58,83): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Bellingshausen[2](58,83): 
workers + city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Bokhara[2](30,79): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Bokhara[2](30,79): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Bokhara[2](30,79): workers + 
city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Scott[3](70,115): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Scott[3](70,115): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Scott[3](70,115): workers + 
city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Sayil[1](99,86): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Tile at Sayil->100,86 marked as unavailable but seems to be available!
1: Failed sanitycheck.c:231 (cityturn.c:282) in Sayil[1](99,86): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Tile at Sayil->100,86 marked as unavailable but seems to be available!
1: Failed sanitycheck.c:334 (cityturn.c:282) in Sayil[1](99,86): workers + 
city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Waxak Tun[2](97,88): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Waxak Tun[2](97,88): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Waxak Tun[2](97,88): workers + 
city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Oudtshoorn[5](122,78): workers 
+ city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Oudtshoorn[5](122,78): workers 
+ city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Oudtshoorn[5](122,78): workers 
+ city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:231 (cityturn.c:282) in Oudtshoorn[5](122,78): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:334 (cityturn.c:282) in Oudtshoorn[5](122,78): workers 
+ city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:240 (cityturn.c:215) in Nieu-Bethesda[3](118,75): 
find_city_from_small_wonder(pplayer, pimprove) == pcity
1: Failed sanitycheck.c:240 (cityturn.c:282) in Nieu-Bethesda[3](118,75): 
find_city_from_small_wonder(pplayer, pimprove) == pcity
1: Failed sanitycheck.c:231 (cityturn.c:215) in Nieu-Bethesda[3](118,75): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:240 (cityturn.c:215) in Nieu-Bethesda[3](118,75): 
find_city_from_small_wonder(pplayer, pimprove) == pcity
1: Failed sanitycheck.c:231 (cityturn.c:282) in Nieu-Bethesda[3](118,75): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitycheck.c:240 (cityturn.c:282) in Nieu-Bethesda[3](118,75): 
find_city_from_small_wonder(pplayer, pimprove) == pcity
1: Failed sanitycheck.c:334 (cityturn.c:282) in Nieu-Bethesda[3](118,75): 
workers + city_specialists(pcity) == pcity->size + 1
1: Failed sanitycheck.c:231 (cityturn.c:215) in Pretoria[3](57,13): 
tile_owner(pcity->tile) == NULL || tile_owner(pcity->tile) == pplayer
1: Failed sanitych

[Freeciv-Dev] (PR#40030) Segfault while drawing map

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40030 >

Thank you for the savegame.  This sounds like the known problem with 
borders that we hope will be solved in 2.2.  I'll use it to test.


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40028) gtk/pango invalid utf8 warning for LANG=fr_FR

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40028 >

Madeline Book wrote:
> Oh wow, it really was trivial! But certainly not obvious
> for the likes of me :(. Once I ran the client with LANG=
> fr_FR.UTF-8 the libc messages displayed correctly and gtk/
> pango was very happy.
> 
Nor to me.  Leaving open until this is documented on our web pages and in
the distributed files.



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


Re: [Freeciv-Dev] (PR#40031) 2.1.2 M$windows title bar not visible in multiple monitor setup

2008-01-21 Thread William Allen Simpson

http://bugs.freeciv.org/Ticket/Display.html?id=40031 >

Shaun Cummings wrote:
> Freeciv 2.1.2 Stable Release (from pre-built Windows binary download)
> 
Which one?  win32?  gtk2?  sdl?


> Windows Vista Ultimate, fully patched as of 1/20/08
> 
> AMD Athlon64 3500+
> 
> 2GB System RAM
> 
> Nvidia GeForce 8600 GT 256MB
> 
> Acer AL2416W 24" LCD (Primary)
> 
> Samsung SyncMaster 710N 17" LCD (Secondary)
> 
Thank you for your unusual example layout, the M$windows screen setup has
been a perennial problem.



___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev