Branch: refs/heads/master
  Commit: 814d1527de2a22040b478afbaf840636b33a80e8
  Author: Cyp <>
  Date:   2012-01-12 (Thu, 12 Jan 2012)

  Changed paths:
    M src/order.cpp

  Log Message:
  Fix dead pointer cleanup in order lists, fixes desynch on manually picking up 

Only a specific list of order types were cleaned up. The list did not contain
DORDER_RECOVER, so queueing oil barrel pickup would result in undefined 
(which includes desynchs). Queueing pickup of the same oil barrel twice would
result in valgrind errors.

Now cleans up all dead pointers, without checking which type of order it is.

Introduced by allowing queueing DORDER_RECOVER in 
but it didn't really make sense to check the order type, since a dead pointer 
is a dead
pointer, independently of order type. Since the order queue no longer abuses a 
void * to
hold a BASE_OBJECT * or STRUCTURE_STATS * depending on order type, the check 
could be

RSA(R) Conference 2012
Mar 27 - Feb 2
Save $400 by Jan. 27
Register now!
Warzone2100-commits mailing list

Reply via email to