Update of patch #4079 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4079 (project freeciv):
Assigned to: cazfi = sveinung
___
Reply to this item at:
http://gna.org/patch/?4079
___
Message sent
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
Update of patch #4079 (project freeciv):
Status:None = Ready For Test
Assigned to:None = cazfi
___
Reply to this item at:
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
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)
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
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
Update of patch #4079 (project freeciv):
Category:None = general
Planned Release: = 2.6.0
___
Follow-up Comment #1:
- by poisoned - by
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
10 matches
Mail list logo
