Comment #7 on issue 185 by nlskrg: Memory leak - PActivityScheduler keeps processed activities in reference

The patch renders the first call of processingActivities.clear(); redundant (in the same procedure the patch was applied to). I think the first line thus should be removed to keep the code clear. Furthermore the example uses piccolo from a thread other than the event dispatch thread. This possibly does not affect the issue demonstrated by the example, but is not the right way to go as piccolo is not thread-safe.

        issue185fix.patch  2.1 KB

Piccolo2D Developers Group:

Reply via email to