URL:
  <http://gna.org/bugs/?20519>

                 Summary: Unloading problem in release builds
                 Project: Freeciv
            Submitted by: cazfi
            Submitted on: Sun 17 Feb 2013 12:02:26 PM EET
                Category: general
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: Ready For Test
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
                 Release: 
         Discussion Lock: Any
        Operating System: None
         Planned Release: 2.4.0, 2.5.0

    _______________________________________________________

Details:

Functionality to remove cargo from the list of transporter's transported units
is inside fc_assert() so it won't happen at all when asserts are disabled.

Fix attached

I'm worried. The fact that there's some such cases of functionality inside
fc_assert() indicates that someone has not been aware of its dangers and could
have made same mistake in many places.
I'll see if CodingStyle needs updating in this respect.



    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sun 17 Feb 2013 12:02:26 PM EET  Name: ReleaseTransportedList.patch 
Size: 904B   By: cazfi

<http://gna.org/bugs/download.php?file_id=17234>

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?20519>

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


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

Reply via email to