Re: [Freeciv-Dev] (PR#40384) [Patch] Fix altering movement crash due to illegal phase

2008-07-19 Thread Jason Dorje Short
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 On 7/19/08, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 With altering movement phase can end being 32, causing crash in client update_info_label(). Fix attached. I'm pretty

Re: [Freeciv-Dev] (PR#40384) [Patch] Fix altering movement crash due to illegal phase

2008-07-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 2008/7/19 Jason Dorje Short [EMAIL PROTECTED]: URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 On 7/19/08, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 With

Re: [Freeciv-Dev] (PR#40384) [Patch] Fix altering movement crash due to illegal phase

2008-07-19 Thread Jason Dorje Short
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 On 7/19/08, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 2008/7/19 Jason Dorje Short [EMAIL PROTECTED]: URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 On 7/19/08,

Re: [Freeciv-Dev] (PR#40384) [Patch] Fix altering movement crash due to illegal phase

2008-07-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40384 2008/7/20 Jason Dorje Short: It's only illegal in that particular assertion. Fix the assertion and that is fixed! Though for backwards compatibility with old clients 2.1 may prefer a workaround. What assertion you are talking