[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Update of patch #4079 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Update of patch #4079 (project freeciv): Assigned to: cazfi = sveinung ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Follow-up Comment #7, patch #4079 (project freeciv): Civ1 and Alien don't use the spy flag. See patch #4078 comment #9 (file #18795) ___ Additional Item Attachment: File name: poison_size_req.patch Size:11 KB ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Update of patch #4079 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Follow-up Comment #5, patch #4079 (project freeciv): Only reading the patch it seems to me that you might call diplomat_escape_full() passing pcity pointer that no longer be valid as city has been destroyed. There seems to be changes not to use pcity inside diplomat_escape_full(), but if it's not needed at all, it should be removed from the parameter list of the function too. ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Follow-up Comment #6, patch #4079 (project freeciv): It is used once - to see if it is NULL or not. I planned to change it to a bool after I had seen that my minimal modifications worked. I forgot. As the code was unchanged reading the patch didn't remind me. Fixed. (file #18772) ___ Additional Item Attachment: File name: poison_size_req.patch Size:13 KB ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Follow-up Comment #3, patch #4079 (project freeciv): Update to apply on top of patch #4078 (file #18715) ___ Additional Item Attachment: File name: poison_size_req.patch Size:11 KB ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Follow-up Comment #2, patch #4079 (project freeciv): Thank you. Fixed. (file #18696) ___ Additional Item Attachment: File name: poison_size_req.patch Size:11 KB ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
Update of patch #4079 (project freeciv): Category:None = general Planned Release: = 2.6.0 ___ Follow-up Comment #1: - by poisoned - by poisoning ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #4079] Minimum size requirement of Poison city to action enablers
URL: http://gna.org/patch/?4079 Summary: Minimum size requirement of Poison city to action enablers Project: Freeciv Submitted by: sveinung Submitted on: Thu 08 Aug 2013 07:53:36 PM GMT Category: None Priority: 5 - Normal Status: None Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: ___ Details: Move the minimum size requirement of the spy action Poison city to action enablers in the rule set. Make Poison city work for a city of size one (if the action is enabled). ___ File Attachments: --- Date: Thu 08 Aug 2013 07:53:36 PM GMT Name: poison_size_req.patch Size: 12kB By: sveinung http://gna.org/patch/download.php?file_id=18657 ___ Reply to this item at: http://gna.org/patch/?4079 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
