UFO: Alien Invasion Issue Tracker
UFO: Alien Invasion
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png UFO: Alien Invasion / Closed Submit Patch #4821 Fix testall crash by resetting particles
Go to the next issue (open or closed)
Go to the next open issue
This issue has been closed with status "Closed" and resolution "Not determined".
Issue basics
  • Type of issue
    Submit Patch
  • Category
    General
  • Targetted for
    Not determined
  • Status
    Closed
  • Priority
    1. Very low
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (0)
There are no items
People involved
Times and dates
  • Posted at
  • Last updated
Issue details
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
[http://sourceforge.net/p/ufoai/patches/694 Item 694] imported from sourceforge.net tracker on 2013-01-28 20:46:59

The tests never cleared the particles, reloading the same ones
multiple times.

When this happpens, a duplicate particle isn't defined, but the old one is
updated with a new set of particle commands. Eventually the particle
command array runs out of room.
===== Comments Ported from Sourceforge =====

====== tlh2000 (2012-10-22 16:45:24.522000) ======

this should be done in test_shared.cpp - i will do this and would be glad about feedback.
====== drane (2012-10-22 16:50:19.362000) ======

ACKed by me.

I saw that test_shared.cpp's include list was VERY clean, whereas the per-test ones weren't. That made me think that the per-test place was the right place to put them

But putting into test_shared does protect ALL of the tests, present and future. Great.
Todos (0 / 0)
Issue created
footer_logo.png The Bug Genie 4.3.1 | Support | Feedback spinning_16.gif