[Freeciv-Dev] (PR#40735) [Patch] map_get_player_city() cleanup

2009-02-23 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=40735 > Since previous vision site cleanup removed vision site references from tiles where city is not, checks in map_get_player_city() are unnecessary. Attached patch removes some of them, and changes some of them into asserts. Now only difference

[Freeciv-Dev] (PR#40610) Re: (PR#40720) cityturn.c:276: auto_arrange_workers: Assertion `cmr.found_a_valid' failed.

2009-02-23 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=40610 > 2009/2/14 Chris Snook: > > I'm using freeciv-2.1.8-1.fc10.i386 from Fedora 10, and it disconnects > from the server (because the server is crashing) as soon as I make a > city, and sometimes when exploring a village (presumably when that > vi

[Freeciv-Dev] (PR#40610) auto_arrange_workers: Assertion `cmr.found_a_valid' failed.

2009-02-23 Thread
http://bugs.freeciv.org/Ticket/Display.html?id=40610 > Thank you very much for your efforts, Chris. Unfortunately, this patch did not work for me. I get the same error. Building with -fno-inline-small-functions (as reported by someone else) does fix the problem for me, if that is helpful in de

[Freeciv-Dev] (PR#40737) [Patch] Fix some compile warnings in S2_0

2009-02-23 Thread Madeline Book
http://bugs.freeciv.org/Ticket/Display.html?id=40737 > Fix the "O_CREAT 3-argument open" warning as reported by Gorillaz on the forum, and replace an alloc macro in the alsa sound code by equivalent code, since it was causing a gcc warning: audio_alsa.c: In function ‘set_hw_params’: audio_alsa.c

[Freeciv-Dev] (PR#40738) [Patch] Fix missing Q_() in connection list column

2009-02-23 Thread Madeline Book
http://bugs.freeciv.org/Ticket/Display.html?id=40738 > It turns out that there was only one place where I used _() instead of Q_(); the attached patch fixes it. Perhaps the fact that Q_() is not necessary in ruleset files should be added to the Internationalization wiki page? -