thirdparty/scummvm
2
0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

RECORDER: Defer mutex deletion to destructor.

Browse source 
Since we now call deinit after a game has been run, otherwise RTL would be
broken. See bug report #3388982 "valgrind errors when doing multiple RTL".
This commit is contained in:
Johannes Schickel 2011-08-09 15:04:26 +02:00
parent a72ab2f448
commit 00d25a77a9
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
common/EventRecorder.cpp
View file

@ -133,6 +133,9 @@ EventRecorder::EventRecorder() {
EventRecorder::~EventRecorder() { EventRecorder::~EventRecorder() {
deinit(); deinit();
g_system->deleteMutex(_timeMutex);
g_system->deleteMutex(_recorderMutex);
} }
void EventRecorder::init() { void EventRecorder::init() {
@ -282,9 +285,6 @@ void EventRecorder::deinit() {
//TODO: remove recordTempFileName'ed file //TODO: remove recordTempFileName'ed file
} }
g_system->deleteMutex(_timeMutex);
g_system->deleteMutex(_recorderMutex);
} }
void EventRecorder::registerRandomSource(RandomSource &rnd, const String &name) { void EventRecorder::registerRandomSource(RandomSource &rnd, const String &name) {