From 23c4e2059649d2a515cde177c8c6bf8eace2cac1 Mon Sep 17 00:00:00 2001
From: Treeki <treeki@gmail.com>
Date: Thu, 6 Dec 2012 23:37:19 +0100
Subject: map music improvements and fixes

---
 src/koopatlas/mapmusic.cpp | 39 ++++++++++++++++++++++++++++-----------
 1 file changed, 28 insertions(+), 11 deletions(-)

(limited to 'src')

diff --git a/src/koopatlas/mapmusic.cpp b/src/koopatlas/mapmusic.cpp
index 8fdf264..9998954 100644
--- a/src/koopatlas/mapmusic.cpp
+++ b/src/koopatlas/mapmusic.cpp
@@ -24,15 +24,20 @@ void dKPMusic::play(int id) {
 	if (s_playing) {
 		// Switch track
 		OSReport("Trying to switch to song %d (Current one is %d)...\n", id, s_song);
-		if (s_song == id || s_nextSong == id)
-			return;
-		if (s_countdownToSwitch >= 0 || s_countdownToFadeIn >= 0)
-			return;
-		OSReport("Will switch; Fading out current track 2 over %d frames\n", FADE_OUT_LEN);
-
-		s_handle.SetTrackVolume(1<<1, FADE_OUT_LEN, 0.0f);
-		s_nextSong = id;
-		s_countdownToSwitch = FADE_OUT_LEN;
+		if (s_song == id || s_nextSong == id) {
+			OSReport("This song is already playing or is scheduled. Not gonna do that.\n");
+		} else if (s_countdownToFadeIn >= 0) {
+			OSReport("There's already a track being faded in (CountdownToFadeIn=%d)\n", s_countdownToFadeIn);
+		} else if (s_countdownToSwitch >= 0) {
+			OSReport("We were already going to switch tracks, but the rstm hasn't been changed yet, so the next song is being changed to this one\n");
+			s_nextSong = id;
+		} else {
+			OSReport("Will switch; Fading out current track 2 over %d frames\n", FADE_OUT_LEN);
+
+			s_handle.SetTrackVolume(1<<1, FADE_OUT_LEN, 0.0f);
+			s_nextSong = id;
+			s_countdownToSwitch = FADE_OUT_LEN;
+		}
 
 	} else {
 		// New track
@@ -66,13 +71,14 @@ void dKPMusic::execute() {
 	if (!s_playing)
 		return;
 
+	if (s_handle.GetSound() == 0)
+		OSReport("SOUND IS NOT PLAYING!\n");
+
 	if (s_countdownToSwitch >= 0) {
 		s_countdownToSwitch--;
 		if (s_countdownToSwitch == 0) {
 			OSReport("Switching brstm files to song %d.\n", s_nextSong);
 
-			s_countdownToFadeIn = BUFFER_CLEAR_DELAY;
-
 			char brstmPath[48];
 			sprintf(brstmPath, "/Sound/new/map%d.er", s_nextSong);
 
@@ -81,7 +87,18 @@ void dKPMusic::execute() {
 			u8 **fileStreamPointer = (u8**)(player+0x808);
 			u8 *fileStream = *fileStreamPointer;
 			DVDHandle *fileInfo = (DVDHandle*)(fileStream+0x28);
+
+			if (fileInfo->unk4 == 1) {
+				OSReport("Was reading chunk, will try again next frame...\n");
+				s_countdownToSwitch++;
+				return;
+			}
+
+			if (s_nextSong > 0)
+				s_countdownToFadeIn = BUFFER_CLEAR_DELAY;
+
 			DVDCancel(fileInfo);
+			//OSReport("CANCEL successfully called!\n");
 			bool result = DVDFastOpen(DVDConvertPathToEntrynum(brstmPath), fileInfo);
 
 			OSReport("StrmSound is at %p, StrmPlayer is at %p, FileStream pointer is at %p, FileStream is at %p, FileInfo is at %p\n", sound, player, fileStreamPointer, fileStream, fileInfo);
-- 
cgit v1.2.3