Author: sveinung
Date: Mon Jul 13 15:54:09 2015
New Revision: 29215
URL: http://svn.gna.org/viewcvs/freeciv?rev=29215&view=rev
Log:
Comment: Explain why a dying unit shouldn't be bounced.
Modified:
trunk/server/citytools.c
Modified: trunk/server/citytools.c
URL:
http://svn.gna.org/viewcvs/freeciv/trunk/server/citytools.c?rev=29215&r1=29214&r2=29215&view=diff
==============================================================================
--- trunk/server/citytools.c (original)
+++ trunk/server/citytools.c Mon Jul 13 15:54:09 2015
@@ -686,6 +686,13 @@
if (vunit->server.dying) {
/* Don't transfer or bounce a dying unit. It will soon be gone
* anyway.
+ *
+ * Bouncing a dying unit isn't a good idea.
+ * Remaining death handling may do things meant for its current
+ * location to the new location. (Example: Stack death)
+ * bounce_unit() will wipe the unit if it can't be bounced. Wiping
+ * the dying unit isn't a good idea. The remaining death handling
+ * code will try to read from it.
*
* Transfering a dying GameLoss unit as part of the loot for
* killing it caused gna bug #23676. */
_______________________________________________
Freeciv-commits mailing list
Freeciv-commits@gna.org
https://mail.gna.org/listinfo/freeciv-commits
