Update of bug #16592 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #16592 (project freeciv):
Assigned to:None = pepeto
Planned Release: = 2.2.3, 2.3.0
___
Follow-up Comment #8:
Attached a very
Update of bug #16592 (project freeciv):
Status:None = Confirmed
___
Follow-up Comment #1:
Backtrace:
(gdb) bt
#0 0x729b5a75 in *__GI_raise (sig=value optimised out)
at
Update of bug #16592 (project freeciv):
Category: client-gtk-2.0 = general
___
Follow-up Comment #2:
French removes shared vision at the turn change. It seems the units in the
V_INVIS vision layer
Follow-up Comment #3, bug #16592 (project freeciv):
That client side assert has revealed at least 5 server side bugs over the
years, so it is very useful if not elegant.
___
Reply to this item at:
http://gna.org/bugs/?16592
Follow-up Comment #4, bug #16592 (project freeciv):
I would suggest to assert there only if 2.2 is configured with --enable-debug
is enabled. Else, it will crash any normal client which connect an old server.
For 2.3, using fc_assert() seems good to prevent the client to crash in
classic
Follow-up Comment #5, bug #16592 (project freeciv):
Reason of the problem:
remove_shared_vision() uses vision_layer_iterate macro which iterates V_MAIN
then V_INVIS. When V_MAIN tile seen count is 0, the tile is fogged, whereas
it didn't handle the V_INVIS units yet.
It should iterate the
Update of bug #16592 (project freeciv):
Status: Confirmed = Ready For Test
___
Follow-up Comment #6:
Fix attached:
* Always remove invisible units before fogging the tile. It was possible to