Placid Report post Posted December 4, 2007 Well, after several days of battling, I've backed my way out of a wholesale rewrite of the actors code and settled for some targeted bug fixing. As this was a smaller change then I was thinking, I've committed to CVS directly. My commit fixes the map change crash related to sounds and adds actors mutex locks to the bits of sound executed in a thread. There are also a few extra checks for NULL pointers and some extra MUTEX_DEBUG code. Sorry for the delay. Nice work, bluap. Do your fixes relate to any of the following bugs: http://el.beplacid.net/bugs/view.php?id=15 http://el.beplacid.net/bugs/view.php?id=12 Share this post Link to post Share on other sites
bluap Report post Posted December 4, 2007 (edited) Do your fixes relate to any of the following bugs: http://el.beplacid.net/bugs/view.php?id=15 http://el.beplacid.net/bugs/view.php?id=12 The first sounds like it is related because of the mutex but I can't see how. However, the problem code could have gone off doing all kinds of nasty stuff so I can't be ruled out. When someone gets round to investigating further, a memory corruption issue might prove a link. The second, I believe is related to KarenRei code that was not updated with some other change. The code is not a release #define. I've added the original forum threads to Mantis. edit: fix it to say what I meant! Edited December 4, 2007 by bluap Share this post Link to post Share on other sites
Placid Report post Posted December 4, 2007 Thanks bluap. I'll leave them for someone who's able to investigate. Share this post Link to post Share on other sites