From 3ce7aaa6d94d81f8456070d78a2de9b55acc2b8c Mon Sep 17 00:00:00 2001
From: Paul Gilbert
Date: Thu, 13 Aug 2015 20:01:35 -0400
Subject: SHERLOCK: RT: Removal of redundant music code

---
 engines/sherlock/music.cpp                 | 23 +++--------------------
 engines/sherlock/music.h                   |  2 --
 engines/sherlock/scalpel/scalpel_darts.cpp | 10 ++--------
 3 files changed, 5 insertions(+), 30 deletions(-)

(limited to 'engines')

diff --git a/engines/sherlock/music.cpp b/engines/sherlock/music.cpp
index 49c48126f4..ccaa6a4b86 100644
--- a/engines/sherlock/music.cpp
+++ b/engines/sherlock/music.cpp
@@ -412,15 +412,6 @@ bool Music::playMusic(const Common::String &name) {
 		stream->read(midiMusicData, midiMusicDataSize);
 		delete stream;
 
-		// for dumping the music tracks
-#if 0
-		Common::DumpFile outFile;
-		outFile.open(name + ".RAW");
-		outFile.write(data, stream->size());
-		outFile.flush();
-		outFile.close();
-#endif
-
 		if (midiMusicDataSize < 14) {
 			warning("Music: not enough data in music file");
 			delete[] midiMusicData;
@@ -502,6 +493,8 @@ bool Music::playMusic(const Common::String &name) {
 		}
 		_mixer->playStream(Audio::Mixer::kMusicSoundType, &_digitalMusicHandle, musicStream);
 	}
+
+	_musicPlaying = true;
 	return true;
 }
 
@@ -510,12 +503,7 @@ void Music::stopMusic() {
 }
 
 void Music::startSong() {
-	if (!_musicOn)
-		return;
-
-	// TODO
-	warning("TODO: Sound::startSong");
-	_musicPlaying = true;
+	// No implementation needed for ScummVM
 }
 
 void Music::freeSong() {
@@ -530,11 +518,6 @@ void Music::freeSong() {
 	_musicPlaying = false;
 }
 
-void Music::waitTimerRoland(uint time) {
-	// TODO
-	warning("TODO: Sound::waitTimerRoland");
-}
-
 bool Music::isPlaying() {
 	if (!IS_3DO) {
 		// MIDI based
diff --git a/engines/sherlock/music.h b/engines/sherlock/music.h
index 93c565c162..66057050e0 100644
--- a/engines/sherlock/music.h
+++ b/engines/sherlock/music.h
@@ -112,8 +112,6 @@ public:
 	 */
 	void stopMusic();
 	
-	void waitTimerRoland(uint time);
-
 	bool isPlaying();
 	uint32 getCurrentPosition();
 
diff --git a/engines/sherlock/scalpel/scalpel_darts.cpp b/engines/sherlock/scalpel/scalpel_darts.cpp
index f36de0915a..2ec86bf7fb 100644
--- a/engines/sherlock/scalpel/scalpel_darts.cpp
+++ b/engines/sherlock/scalpel/scalpel_darts.cpp
@@ -382,10 +382,7 @@ int Darts::doPowerBar(const Common::Point &pt, byte color, int goToPower, bool i
 	int idx = 0;
 
 	events.clearEvents();
-	if (music._musicOn)
-		music.waitTimerRoland(10);
-	else
-		events.delay(100);
+	events.delay(100);
 
 	// Display loop
 	do {
@@ -410,10 +407,7 @@ int Darts::doPowerBar(const Common::Point &pt, byte color, int goToPower, bool i
 			screen.slamArea(pt.x + idx, pt.y, 1, 8);
 		}
 
-		if (music._musicOn) {
-			if (!(idx % 3))
-				music.waitTimerRoland(1);
-		} else if (!(idx % 8))
+		if (!(idx % 8))
 			events.wait(1);
 	
 		++idx;
-- 
cgit v1.2.3