Re: [Freeciv-Dev] (PR#39686) [Bug] Unit pointer used after move_unit()

2007-09-10 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=39686 > On 08/09/2007, Marko Lindqvist wrote: > > move_unit() ... > Now some > callers are ready to segfault when ever moving unit dies due to > autoattack or by entering barbarian hut (or some other reason?). If nothing comes up in further t

Re: [Freeciv-Dev] (PR#39686) [Bug] Unit pointer used after move_unit()

2007-09-09 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=39686 > On 09/09/07, Marko Lindqvist wrote: > > On 08/09/07, Marko Lindqvist wrote: > > > > move_unit() > > ... > > > Now some > > callers are ready to segfault when ever moving unit dies due to > > autoattack or by entering barbarian hut (or som

Re: [Freeciv-Dev] (PR#39686) [Bug] Unit pointer used after move_unit()

2007-09-09 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=39686 > On 08/09/07, Marko Lindqvist wrote: > > move_unit() ... > Now some > callers are ready to segfault when ever moving unit dies due to > autoattack or by entering barbarian hut (or some other reason?). Minimally tested patch for S2_1.

[Freeciv-Dev] (PR#39686) [Bug] Unit pointer used after move_unit()

2007-09-08 Thread Marko Lindqvist
http://bugs.freeciv.org/Ticket/Display.html?id=39686 > It has been bad idea to remove return value from move_unit() in trunk. Even if it was not used by callers, it should be. Now some callers are ready to segfault when ever moving unit dies due to autoattack or by entering barbarian hut (or som