[Scummvm-cvs-logs] SF.net SVN: scummvm:[49098] scummvm/trunk/engines/sci

m_kiewitz at users.sourceforge.net m_kiewitz at users.sourceforge.net
Wed May 19 16:19:16 CEST 2010


Revision: 49098
          http://scummvm.svn.sourceforge.net/scummvm/?rev=49098&view=rev
Author:   m_kiewitz
Date:     2010-05-19 14:19:16 +0000 (Wed, 19 May 2010)

Log Message:
-----------
SCI: cleanup soundcode - number renamed to resourceId, prio renamed to priority

Modified Paths:
--------------
    scummvm/trunk/engines/sci/engine/savegame.cpp
    scummvm/trunk/engines/sci/sound/music.cpp
    scummvm/trunk/engines/sci/sound/music.h
    scummvm/trunk/engines/sci/sound/soundcmd.cpp

Modified: scummvm/trunk/engines/sci/engine/savegame.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/savegame.cpp	2010-05-19 13:32:27 UTC (rev 49097)
+++ scummvm/trunk/engines/sci/engine/savegame.cpp	2010-05-19 14:19:16 UTC (rev 49098)
@@ -112,8 +112,8 @@
 		uint32 handle = 0;
 		s.syncAsSint32LE(handle);
 		soundObj = DEFROBNICATE_HANDLE(handle);
-		s.syncAsSint32LE(resnum);
-		s.syncAsSint32LE(prio);
+		s.syncAsSint32LE(resourceId);
+		s.syncAsSint32LE(priority);
 		s.syncAsSint32LE(status);
 		s.skip(4);	// restoreBehavior
 		uint32 restoreTime = 0;
@@ -133,11 +133,11 @@
 	} else {
 		// A bit more optimized saving
 		sync_reg_t(s, soundObj);
-		s.syncAsSint16LE(resnum);
+		s.syncAsSint16LE(resourceId);
 		s.syncAsSint16LE(dataInc);
 		s.syncAsSint16LE(ticker);
 		s.syncAsSint16LE(signal, VER(17));
-		s.syncAsByte(prio);
+		s.syncAsByte(priority);
 		s.syncAsSint16LE(loop, VER(17));
 		s.syncAsByte(volume);
 		s.syncAsByte(hold, VER(17));

Modified: scummvm/trunk/engines/sci/sound/music.cpp
===================================================================
--- scummvm/trunk/engines/sci/sound/music.cpp	2010-05-19 13:32:27 UTC (rev 49097)
+++ scummvm/trunk/engines/sci/sound/music.cpp	2010-05-19 14:19:16 UTC (rev 49098)
@@ -171,7 +171,7 @@
 }
 
 static int f_compare(const void *arg1, const void *arg2) {
-	return ((const MusicEntry *)arg2)->prio - ((const MusicEntry *)arg1)->prio;
+	return ((const MusicEntry *)arg2)->priority - ((const MusicEntry *)arg1)->priority;
 }
 
 void SciMusic::sortPlayList() {
@@ -302,7 +302,7 @@
 void SciMusic::soundSetPriority(MusicEntry *pSnd, byte prio) {
 	Common::StackLock lock(_mutex);
 
-	pSnd->prio = prio;
+	pSnd->priority = prio;
 	sortPlayList();
 }
 
@@ -392,7 +392,7 @@
 	for (uint32 i = 0; i < _playList.size(); i++) {
 		MusicEntry *song = _playList[i];
 		con->DebugPrintf("%d: %04x:%04x, resource id: %d, status: %s, %s type\n", i,
-						PRINT_REG(song->soundObj), song->resnum,
+						PRINT_REG(song->soundObj), song->resourceId,
 						musicStatus[song->status], song->pMidiParser ? "MIDI" : "digital audio");
 	}
 }
@@ -406,9 +406,9 @@
 	for (MusicList::iterator i = _playList.begin(); i != end; ++i) {
 		MusicEntry *song = *i;
 		if (song->soundObj == obj) {
-			con->DebugPrintf("Resource id: %d, status: %s\n", song->resnum, musicStatus[song->status]);
+			con->DebugPrintf("Resource id: %d, status: %s\n", song->resourceId, musicStatus[song->status]);
 			con->DebugPrintf("dataInc: %d, hold: %d, loop: %d\n", song->dataInc, song->hold, song->loop);
-			con->DebugPrintf("signal: %d, priority: %d\n", song->signal, song->prio);
+			con->DebugPrintf("signal: %d, priority: %d\n", song->signal, song->priority);
 			con->DebugPrintf("ticker: %d, volume: %d\n", song->ticker, song->volume);
 
 			if (song->pMidiParser) {
@@ -442,12 +442,12 @@
 	soundObj = NULL_REG;
 
 	soundRes = 0;
-	resnum = 0;
+	resourceId = 0;
 
 	dataInc = 0;
 	ticker = 0;
 	signal = 0;
-	prio = 0;
+	priority = 0;
 	loop = 0;
 	volume = MUSIC_VOLUME_DEFAULT;
 	hold = 0;

Modified: scummvm/trunk/engines/sci/sound/music.h
===================================================================
--- scummvm/trunk/engines/sci/sound/music.h	2010-05-19 13:32:27 UTC (rev 49097)
+++ scummvm/trunk/engines/sci/sound/music.h	2010-05-19 14:19:16 UTC (rev 49098)
@@ -69,12 +69,12 @@
 	reg_t soundObj;
 
 	SoundResource *soundRes;
-	uint16 resnum;
+	uint16 resourceId;
 
 	uint16 dataInc;
 	uint16 ticker;
 	uint16 signal;
-	byte prio;
+	byte priority;
 	uint16 loop;
 	int16 volume;
 	byte hold;

Modified: scummvm/trunk/engines/sci/sound/soundcmd.cpp
===================================================================
--- scummvm/trunk/engines/sci/sound/soundcmd.cpp	2010-05-19 13:32:27 UTC (rev 49097)
+++ scummvm/trunk/engines/sci/sound/soundcmd.cpp	2010-05-19 14:19:16 UTC (rev 49098)
@@ -253,7 +253,7 @@
 	if (!obj.segment)
 		return;
 
-	int number = GET_SEL32V(_segMan, obj, SELECTOR(number));
+	int resourceId = GET_SEL32V(_segMan, obj, SELECTOR(number));
 
 #ifdef USE_OLD_MUSIC_FUNCTIONS
 
@@ -273,10 +273,10 @@
 		}
 	}
 
-	if (!obj.segment || !_resMan->testResource(ResourceId(kResourceTypeSound, number)))
+	if (!obj.segment || !_resMan->testResource(ResourceId(kResourceTypeSound, resourceId)))
 		return;
 
-	_state->sfx_add_song(build_iterator(_resMan, number, type, handle), 0, handle, number);
+	_state->sfx_add_song(build_iterator(_resMan, resourceId, type, handle), 0, handle, resourceId);
 
 
 	// Notify the engine
@@ -295,15 +295,15 @@
 		cmdDisposeSound(obj, value);
 
 	MusicEntry *newSound = new MusicEntry();
-	newSound->resnum = number;
-	if (number && _resMan->testResource(ResourceId(kResourceTypeSound, number)))
-		newSound->soundRes = new SoundResource(number, _resMan, _soundVersion);
+	newSound->resourceId = resourceId;
+	if (resourceId && _resMan->testResource(ResourceId(kResourceTypeSound, resourceId)))
+		newSound->soundRes = new SoundResource(resourceId, _resMan, _soundVersion);
 	else
 		newSound->soundRes = 0;
 
 	newSound->soundObj = obj;
 	newSound->loop = GET_SEL32V(_segMan, obj, SELECTOR(loop));
-	newSound->prio = GET_SEL32V(_segMan, obj, SELECTOR(pri)) & 0xFF;
+	newSound->priority = GET_SEL32V(_segMan, obj, SELECTOR(pri)) & 0xFF;
 	if (_soundVersion >= SCI_VERSION_1_LATE)
 		newSound->volume = CLIP<int>(GET_SEL32V(_segMan, obj, SELECTOR(vol)), 0, MUSIC_VOLUME_MAX);
 
@@ -312,10 +312,10 @@
 	// effects. If the resource exists, play it using map 65535 (sound
 	// effects map)
 
-	if (getSciVersion() >= SCI_VERSION_1_1 && _resMan->testResource(ResourceId(kResourceTypeAudio, number))) {
+	if (getSciVersion() >= SCI_VERSION_1_1 && _resMan->testResource(ResourceId(kResourceTypeAudio, resourceId))) {
 		// Found a relevant audio resource, play it
 		int sampleLen;
-		newSound->pStreamAud = _audio->getAudioStream(number, 65535, &sampleLen);
+		newSound->pStreamAud = _audio->getAudioStream(resourceId, 65535, &sampleLen);
 		newSound->soundType = Audio::Mixer::kSpeechSoundType;
 	} else {
 		if (newSound->soundRes)
@@ -415,16 +415,16 @@
 		return;
 	}
 
-	int number = obj.segment ? GET_SEL32V(_segMan, obj, SELECTOR(number)) : -1;
+	int resourceId = obj.segment ? GET_SEL32V(_segMan, obj, SELECTOR(number)) : -1;
 
-	if (musicSlot->resnum != number) { // another sound loaded into struct
+	if (musicSlot->resourceId != resourceId) { // another sound loaded into struct
 		cmdDisposeSound(obj, value);
 		cmdInitSound(obj, value);
 		// Find slot again :)
 		musicSlot = _music->getSlot(obj);
 	}
 	int16 loop = GET_SEL32V(_segMan, obj, SELECTOR(loop));
-	debugC(2, kDebugLevelSound, "cmdPlaySound: resource number %d, loop %d", number, loop);
+	debugC(2, kDebugLevelSound, "cmdPlaySound: resource number %d, loop %d", resourceId, loop);
 
 	PUT_SEL32(_segMan, obj, SELECTOR(handle), obj);
 
@@ -439,7 +439,7 @@
 	}
 
 	musicSlot->loop = GET_SEL32V(_segMan, obj, SELECTOR(loop));
-	musicSlot->prio = GET_SEL32V(_segMan, obj, SELECTOR(priority));
+	musicSlot->priority = GET_SEL32V(_segMan, obj, SELECTOR(priority));
 	if (_soundVersion >= SCI_VERSION_1_LATE)
 		musicSlot->volume = GET_SEL32V(_segMan, obj, SELECTOR(vol));
 	_music->soundPlay(musicSlot);
@@ -729,7 +729,7 @@
 	if (objVol != musicSlot->volume)
 		_music->soundSetVolume(musicSlot, objVol);
 	uint32 objPrio = GET_SEL32V(_segMan, obj, SELECTOR(pri));
-	if (objPrio != musicSlot->prio)
+	if (objPrio != musicSlot->priority)
 		_music->soundSetPriority(musicSlot, objPrio);
 
 #endif
@@ -984,7 +984,7 @@
 
 	if (value == -1) {
 		// Set priority from the song data
-		Resource *song = _resMan->findResource(ResourceId(kResourceTypeSound, musicSlot->resnum), 0);
+		Resource *song = _resMan->findResource(ResourceId(kResourceTypeSound, musicSlot->resourceId), 0);
 		if (song->data[0] == 0xf0)
 			_music->soundSetPriority(musicSlot, song->data[1]);
 		else
@@ -1077,8 +1077,8 @@
 
 	const MusicList::iterator end = _music->getPlayListEnd();
 	for (MusicList::iterator i = _music->getPlayListStart(); i != end; ++i) {
-		if ((*i)->resnum && _resMan->testResource(ResourceId(kResourceTypeSound, (*i)->resnum))) {
-			(*i)->soundRes = new SoundResource((*i)->resnum, _resMan, _soundVersion);
+		if ((*i)->resourceId && _resMan->testResource(ResourceId(kResourceTypeSound, (*i)->resourceId))) {
+			(*i)->soundRes = new SoundResource((*i)->resourceId, _resMan, _soundVersion);
 			_music->soundInitSnd(*i);
 		} else {
 			(*i)->soundRes = 0;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list