[Freeciv-Dev] [bug #22277] raze_city() can try to remove improvement twice

[Jacob Nevins]

Update of bug #22277 (project freeciv):

  Status:  Ready For Test => Fixed  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  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] [bug #22277] raze_city() can try to remove improvement twice

[Jacob Nevins]

Update of bug #22277 (project freeciv):

  Status: In Progress => Ready For Test 

___

Additional Item Attachment:

File name: trunk-S2_5-S2_4-raze-city-assertion.patch Size:1 KB


___

Reply to this item at:

  

___
  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] [bug #22277] raze_city() can try to remove improvement twice

[Jacob Nevins]

URL:
  

 Summary: raze_city() can try to remove improvement twice
 Project: Freeciv
Submitted by: jtn
Submitted on: Sat 05 Jul 2014 22:24:54 BST
Category: None
Severity: 2 - Minor
Priority: 5 - Normal
  Status: In Progress
 Assigned to: jtn
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: Any
 Planned Release: 2.4.3, 2.5.0, 2.6.0

___

Details:

If raze_city() finds a small wonder, it'll try to remove it twice.

However, as the comment notes, it should never find a small wonder, as that
should have been removed earlier. So this bug has no functional effect, and
the code can be turned into an assertion.




___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev