[Scummvm-cvs-logs] scummvm master -> 9ca6562dc60a769199a3085beab0a72b72382deb

Strangerke arnaud.boutonne at gmail.com
Fri Apr 15 20:38:35 CEST 2011


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
9ca6562dc6 TSAGE: Cleanup. For coding consistency, remove default parameters when calling startSound() in scene groups 8 and 10


Commit: 9ca6562dc60a769199a3085beab0a72b72382deb
    https://github.com/scummvm/scummvm/commit/9ca6562dc60a769199a3085beab0a72b72382deb
Author: strangerke (arnaud.boutonne at gmail.com)
Date: 2011-04-15T11:36:54-07:00

Commit Message:
TSAGE: Cleanup. For coding consistency, remove default parameters when calling startSound() in scene groups 8 and 10

Changed paths:
    engines/tsage/ringworld_scenes10.cpp
    engines/tsage/ringworld_scenes8.cpp



diff --git a/engines/tsage/ringworld_scenes10.cpp b/engines/tsage/ringworld_scenes10.cpp
index 3aba650..4d413a0 100644
--- a/engines/tsage/ringworld_scenes10.cpp
+++ b/engines/tsage/ringworld_scenes10.cpp
@@ -155,7 +155,7 @@ void Scene9100::postInit(SceneObjectList *OwnerList) {
 	_sceneHotspot5.setup(69, 36, 121, 272, 9100, 45, 46);
 	_sceneHotspot6.setup(127, 0, 200, 52, 9100, 47, 48);
 
-	_globals->_soundHandler.startSound(251, 0, 127);
+	_globals->_soundHandler.startSound(251);
 	if (_globals->_sceneManager._previousScene == 9150) {
 		if (_globals->getFlag(20)) {
 			_globals->_player.disableControl();
@@ -240,7 +240,7 @@ void Scene9150::dispatch() {
 		} else {
 			_globals->_player.disableControl();
 			if (_globals->getFlag(11)) {
-				_globals->_soundHandler.startSound(286, 0, 127);
+				_globals->_soundHandler.startSound(286);
 				_sceneMode = 9153;
 			} else {
 				_sceneMode = 9156;
@@ -274,7 +274,7 @@ void Scene9150::postInit(SceneObjectList *OwnerList) {
 	_sceneHotspot8.setup(133, 584, 142, 640, 9150, 57, -1);
 	_sceneHotspot10.setup(83, 304, 103, 323, 9150, 58, 59);
 
-	_globals->_soundHandler.startSound(285, 0, 127);
+	_globals->_soundHandler.startSound(285);
 	_globals->_player.disableControl();
 
 	if (_globals->getFlag(20)) {
@@ -406,7 +406,7 @@ void Scene9200::postInit(SceneObjectList *OwnerList) {
 	_object1.animate(ANIM_MODE_2, 0);
 	_object1.setPosition(Common::Point(132, 114), 0);
 	_object1.setPriority2(140);
-	_soundHandler.startSound(297, 0, 127);
+	_soundHandler.startSound(297);
 	_stripManager.addSpeaker(&_speakerQText);
 	_stripManager.addSpeaker(&_speakerGR);
 	_stripManager.addSpeaker(&_speakerGText);
@@ -479,7 +479,7 @@ void Scene9300::signal() {
 		_globals->setFlag(84);
 		// No break on purpose
 	case 9303:
-		_globals->_soundHandler.startSound(295, 0, 127);
+		_globals->_soundHandler.startSound(295);
 		_globals->_sceneManager.changeScene(9350);
 		break;
 	case 9302:
@@ -509,7 +509,7 @@ void Scene9300::postInit(SceneObjectList *OwnerList) {
 	_globals->_player.changeZoom(-1);
 	_object1.postInit();
 	_object2.postInit();
-	_globals->_soundHandler.startSound(289, 0, 127);
+	_globals->_soundHandler.startSound(289);
 
 	_hotspot1.setup(35, 142, 76, 212, 9300, 0, 1);
 	_hotspot2.setup(28, 90, 81, 143, 9300, 2, 3);
@@ -763,7 +763,7 @@ void Scene9400::signal() {
 void Scene9400::dispatch() {
 	if ((_object1._animateMode == 2) && (_object1._strip == 1) && (_object1._frame == 4)){
 		if (_field1032 == 0) {
-			_soundHandler.startSound(296, 0, 127);
+			_soundHandler.startSound(296);
 			_field1032 = 1;
 		}
 	} else {
@@ -1072,7 +1072,7 @@ void Scene9500::signal() {
 	switch (_sceneMode) {
 	case 9503:
 		_globals->_sceneManager.changeScene(9200);
-		_globals->_soundHandler.startSound(295, 0, 127);
+		_globals->_soundHandler.startSound(295);
 		break;
 	case 9504:
 		_globals->_sceneManager.changeScene(9850);
@@ -1128,7 +1128,7 @@ void Scene9500::postInit(SceneObjectList *OwnerList) {
 	setZoomPercents(110, 75, 200, 150);
 
 	_globals->_player.postInit();
-	_globals->_soundHandler.startSound(305, 0, 127);
+	_globals->_soundHandler.startSound(305);
 
 	_candle.postInit(0);
 	_candle.setVisage(9500);
@@ -1236,7 +1236,7 @@ void Scene9700::signal() {
 		_globals->_events.setCursor(CURSOR_USE);
 		break;
 	case 9704:
-		_globals->_soundHandler.startSound(323, 0, 127);
+		_globals->_soundHandler.startSound(323);
 		_globals->_sceneManager.changeScene(9750);
 		break;
 	}
@@ -1426,7 +1426,7 @@ void Scene9850::Hotspot17::doAction(int action) {
 		SceneItem::display(9850, 32, SET_Y, 20, SET_WIDTH, 200, SET_EXT_BGCOLOR, 7, LIST_END);
 	} else {
 		if (action == CURSOR_USE)
-			scene->_soundHandler.startSound(306, 0, 127);
+			scene->_soundHandler.startSound(306);
 		NamedHotspot::doAction(action);
 	}
 }
@@ -1438,7 +1438,7 @@ void Scene9850::Hotspot18::doAction(int action) {
 		SceneItem::display(9850, 32, SET_Y, 20, SET_WIDTH, 200, SET_EXT_BGCOLOR, 7, LIST_END);
 	} else {
 		if (action == CURSOR_USE)
-			scene->_soundHandler.startSound(306, 0, 127);
+			scene->_soundHandler.startSound(306);
 		NamedHotspot::doAction(action);
 	}
 }
@@ -1450,7 +1450,7 @@ void Scene9850::Hotspot19::doAction(int action) {
 		SceneItem::display(9850, 31, SET_Y, 20, SET_WIDTH, 200, SET_EXT_BGCOLOR, 7, LIST_END);
 	} else {
 		if (action == CURSOR_USE)
-			scene->_soundHandler.startSound(313, 0, 127);
+			scene->_soundHandler.startSound(313);
 		NamedHotspot::doAction(action);
 	}
 }
@@ -1639,7 +1639,7 @@ void Scene9900::strAction1::signal() {
 
 	switch (_actionIndex++) {
 	case 0:
-		scene->_soundHandler.startSound(351, 0, 127);
+		scene->_soundHandler.startSound(351);
 		_object9.postInit();
 		_object9.setVisage(18);
 		_object9._frame = 1;
@@ -1658,7 +1658,7 @@ void Scene9900::strAction1::signal() {
 		_globals->_scenePalette.addFader(&mask2[0], 1, 5, this);
 		break;
 	case 3:
-		_globals->_soundHandler.startSound(377, 0, 127);
+		_globals->_soundHandler.startSound(377);
 		setDelay(120);
 		break;
 	case 4:
@@ -1829,7 +1829,7 @@ void Scene9900::signal() {
 
 	switch (_sceneMode){
 	case 150:
-		_globals->_soundHandler.startSound(380, 0, 127);
+		_globals->_soundHandler.startSound(380);
 		_object8.postInit(0);
 		_object8.setVisage(2002);
 		_object8.setStrip(1);
@@ -1862,7 +1862,7 @@ void Scene9900::signal() {
 		setAction(&_sequenceManager, this, 9902, &_object1, &_object2, &_object3, &_object4, &_object5, &_object6);
 		break;
 	case 9904:
-		_globals->_soundHandler.startSound(390, 0, 127);
+		_globals->_soundHandler.startSound(390);
 		_sceneMode = 9912;
 		setAction(&_strAction2, this);
 		break;
@@ -1893,7 +1893,7 @@ void Scene9900::signal() {
 		setAction(&_sequenceManager, this, 9904, &_object1, &_object2, &_object3, &_object4, &_object5, &_object6);
 		break;
 	case 9909:
-		_globals->_soundHandler.startSound(375, 0, 127);
+		_globals->_soundHandler.startSound(375);
 		_globals->_player.disableControl();
 		_sceneMode = 9907;
 		setAction(&_sequenceManager, this, 9907, &_object1, &_object2, &_object3, &_object4, &_object5, &_object6);
@@ -1904,7 +1904,7 @@ void Scene9900::signal() {
 		setAction(&_sequenceManager, this, 9911, &_object1, &_object2, &_object3, &_object4, &_object5, &_object6);
 		break;
 	case 9911:
-		_globals->_soundHandler.startSound(367, 0, 127);
+		_globals->_soundHandler.startSound(367);
 		_globals->_player.disableControl();
 		_sceneMode = 9909;
 		setAction(&_sequenceManager, this, 9909, &_object1, &_object2, &_object3, &_object4, &_object5, &_object6);
@@ -2055,7 +2055,7 @@ void Scene9999::postInit(SceneObjectList *OwnerList) {
 	else
 		_globals->_stripNum = 2121;
 
-	_globals->_soundHandler.startSound(118, 0, 127);
+	_globals->_soundHandler.startSound(118);
 
 }
 
diff --git a/engines/tsage/ringworld_scenes8.cpp b/engines/tsage/ringworld_scenes8.cpp
index a2742be..35db3cd 100644
--- a/engines/tsage/ringworld_scenes8.cpp
+++ b/engines/tsage/ringworld_scenes8.cpp
@@ -48,7 +48,7 @@ void Scene7000::Action1::signal() {
 		setAction(&scene->_action6, this);
 		break;
 	case 2:
-		scene->_soundHandler.startSound(252, 0, 127);
+		scene->_soundHandler.startSound(252);
 		scene->_object8.remove();
 		scene->_object1.postInit();
 		scene->_object1.setVisage(7003);
@@ -169,7 +169,7 @@ void Scene7000::Action4::signal() {
 		setDelay(300);
 		break;
 	case 2:
-		_globals->_soundHandler.startSound(252, 0, 127);
+		_globals->_soundHandler.startSound(252);
 		scene->_object1.show();
 		scene->_object1.setStrip(3);
 		scene->_object1.setFrame(1);
@@ -199,7 +199,7 @@ void Scene7000::Action5::signal() {
 	}
 	case 1:
 		_globals->_player.checkAngle(&scene->_object1);
-		_globals->_soundHandler.startSound(252, 0, 127);
+		_globals->_soundHandler.startSound(252);
 		scene->_object1.setStrip(2);
 		scene->_stripManager.start(7015, this);
 		break;
@@ -532,7 +532,7 @@ void Scene7000::postInit(SceneObjectList *OwnerList) {
 		_object1.animate(ANIM_MODE_8, 0, 0);
 		_globals->_sceneItems.addItems(&_object1, 0);
 	}
-	_soundHandler.startSound(251, 0, 127);
+	_soundHandler.startSound(251);
 	if (_globals->_sceneManager._previousScene == 2100) {
 		if (_globals->getFlag(72)) {
 			_globals->_player.postInit();
@@ -550,7 +550,7 @@ void Scene7000::postInit(SceneObjectList *OwnerList) {
 				setAction(&_action1);
 			}
 		} else {
-			_globals->_soundHandler.startSound(250, 0, 127);
+			_globals->_soundHandler.startSound(250);
 			_globals->setFlag(72);
 
 			_object3.postInit();
@@ -597,9 +597,9 @@ void Scene7000::postInit(SceneObjectList *OwnerList) {
 		_object3.setVisage(5001);
 		_object3.setStrip2(1);
 		_object3.setPosition(Common::Point(307, 0), 0);
-		_soundHandler.startSound(151, 0, 127);
+		_soundHandler.startSound(151);
 		_soundHandler.proc5(1);
-		_globals->_soundHandler.startSound(250, 0, 127);
+		_globals->_soundHandler.startSound(250);
 		setAction(&_action3);
 	}
 }
@@ -1114,9 +1114,9 @@ void Scene7100::postInit(SceneObjectList *OwnerList) {
 	_object1.setPosition(Common::Point(100, 100), 0);
 
 	setAction(&_action11);
-	_soundHandler1.startSound(270, 0, 127);
-	_soundHandler2.startSound(275, 0, 127);
-	_globals->_soundHandler.startSound(270, 0, 127);
+	_soundHandler1.startSound(270);
+	_soundHandler2.startSound(275);
+	_globals->_soundHandler.startSound(270);
 }
 /*--------------------------------------------------------------------------
  * Scene 7200
@@ -1281,7 +1281,7 @@ void Scene7200::postInit(SceneObjectList *OwnerList) {
 	_swimmer.setPosition(Common::Point(-8, 16), 0);
 
 	setAction(&_action1);
-	_soundHandler.startSound(271, 0, 127);
+	_soundHandler.startSound(271);
 }
 
 /*--------------------------------------------------------------------------
@@ -1476,7 +1476,7 @@ void Scene7300::postInit(SceneObjectList *OwnerList) {
 	_object8._numFrames = 2;
 
 	setAction(&_action1);
-	_globals->_soundHandler.startSound(272, 0, 127);
+	_globals->_soundHandler.startSound(272);
 }
 
 /*--------------------------------------------------------------------------
@@ -1644,7 +1644,7 @@ void Scene7700::Action3::signal() {
 		setDelay(60);
 		// No break on purpose!
 	case 2:
-		scene->_soundHandler.startSound(260, 0, 127);
+		scene->_soundHandler.startSound(260);
 		scene->_object8.setVisage(7703);
 		scene->_object8.setPosition(Common::Point(177, 97), 0);
 		scene->_object8.setStrip2(3);
@@ -1853,7 +1853,7 @@ void Scene7700::SceneHotspot8::doAction(int action) {
 		break;
 	case CURSOR_USE:
 		scene->_sceneMode = 7709;
-		scene->_soundHandler.startSound(259, 0, 127);
+		scene->_soundHandler.startSound(259);
 		scene->_object15.setFrame(scene->_object15.getFrameCount());
 		scene->_object15.animate(ANIM_MODE_6, scene);
 		if ((scene->_field977 == 2) && (scene->_field97B == 0)) {
@@ -1879,7 +1879,7 @@ void Scene7700::SceneHotspot9::doAction(int action) {
 		break;
 	case CURSOR_USE:
 		scene->_sceneMode = 7709;
-		scene->_soundHandler.startSound(259, 0, 127);
+		scene->_soundHandler.startSound(259);
 		scene->_object15.setFrame(1);
 		scene->_object15.animate(ANIM_MODE_5, scene);
 		if (scene->_field977 > 2) {
@@ -2102,7 +2102,7 @@ void Scene7700::Object9::doAction(int action) {
 					_globals->_sceneItems.push_front(&scene->_object10);
 					scene->_object10.setPriority2(240);
 				}
-				scene->_soundHandler.startSound(262, 0, 127);
+				scene->_soundHandler.startSound(262);
 				scene->_object14.animate(ANIM_MODE_5, 0);
 			}
 			_globals->_events.setCursor(CURSOR_WALK);
@@ -2209,7 +2209,7 @@ void Scene7700::signal() {
 		_globals->_player.enableControl();
 		break;
 	case 7704:
-		_globals->_soundHandler.startSound(256, 0, 127);
+		_globals->_soundHandler.startSound(256);
 		_prof.setStrip2(4);
 		_prof.setFrame2(1);
 		_prof.setPosition(Common::Point(159, 87), 0);
@@ -2493,7 +2493,7 @@ void Scene7700::postInit(SceneObjectList *OwnerList) {
 
 	_sceneMode = 7701;
 	setAction(&_sequenceManager, this, 7701, &_globals->_player, 0);
-	_soundHandler.startSound(256, 0, 127);
+	_soundHandler.startSound(256);
 }
 
 Scene7700::Scene7700() {






More information about the Scummvm-git-logs mailing list