aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--scumm/imuse.cpp13
-rw-r--r--scumm/imuse_internal.h3
2 files changed, 12 insertions, 4 deletions
diff --git a/scumm/imuse.cpp b/scumm/imuse.cpp
index 9076cec2d4..8591463891 100644
--- a/scumm/imuse.cpp
+++ b/scumm/imuse.cpp
@@ -316,7 +316,7 @@ int IMuseInternal::stopAllSounds() {
}
void IMuseInternal::on_timer(MidiDriver *midi) {
- if (_paused)
+ if (_paused || !_initialized)
return;
if (midi == _midi_native || !_midi_native)
@@ -612,9 +612,16 @@ int IMuseInternal::get_master_volume() {
return _master_volume;
}
-int IMuseInternal::terminate() {
+int IMuseInternal::terminate1() {
+ _initialized = false;
stopAllSounds();
+ return 0;
+}
+// This is the stuff that has to be done
+// outside the monitor's mutex, otherwise
+// a deadlock occurs.
+int IMuseInternal::terminate2() {
if (_midi_adlib) {
_midi_adlib->close();
delete _midi_adlib;
@@ -1779,7 +1786,7 @@ int32 IMuse::doCommand (int numargs, int args[]) { in(); int32 ret = _target->do
int IMuse::clear_queue() { in(); int ret = _target->clear_queue(); out(); return ret; }
void IMuse::setBase(byte **base) { in(); _target->setBase(base); out(); }
uint32 IMuse::property(int prop, uint32 value) { in(); uint32 ret = _target->property(prop, value); out(); return ret; }
-void IMuse::terminate() { in(); _target->terminate(); out(); }
+void IMuse::terminate() { in(); _target->terminate1(); out(); _target->terminate2(); }
// The IMuse::create method provides a front-end factory
// for creating IMuseInternal without exposing that class
diff --git a/scumm/imuse_internal.h b/scumm/imuse_internal.h
index 0bc88e7a5e..4d74679336 100644
--- a/scumm/imuse_internal.h
+++ b/scumm/imuse_internal.h
@@ -450,7 +450,8 @@ public:
void on_timer(MidiDriver *midi);
void pause(bool paused);
- int terminate();
+ int terminate1();
+ int terminate2();
int save_or_load(Serializer *ser, Scumm *scumm);
int set_music_volume(uint vol);
int get_music_volume();