[Scummvm-git-logs] scummvm master -> 3877a0642b5fa5a9471dda2c9eb25a575fdfc3ae

dreammaster dreammaster at scummvm.org
Mon Apr 17 00:16:26 CEST 2017


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:
3877a0642b TITANIC: Renamings for mover position updates


Commit: 3877a0642b5fa5a9471dda2c9eb25a575fdfc3ae
    https://github.com/scummvm/scummvm/commit/3877a0642b5fa5a9471dda2c9eb25a575fdfc3ae
Author: Paul Gilbert (dreammaster at scummvm.org)
Date: 2017-04-16T18:16:08-04:00

Commit Message:
TITANIC: Renamings for mover position updates

Changed paths:
    engines/titanic/star_control/camera_mover.cpp
    engines/titanic/star_control/camera_mover.h
    engines/titanic/star_control/star_camera.cpp
    engines/titanic/star_control/star_control_sub21.cpp
    engines/titanic/star_control/star_control_sub21.h
    engines/titanic/star_control/star_control_sub22.cpp
    engines/titanic/star_control/star_control_sub22.h


diff --git a/engines/titanic/star_control/camera_mover.cpp b/engines/titanic/star_control/camera_mover.cpp
index 198e21a..ab99a2f 100644
--- a/engines/titanic/star_control/camera_mover.cpp
+++ b/engines/titanic/star_control/camera_mover.cpp
@@ -90,11 +90,11 @@ void CCameraMover::proc7() {
 	}
 }
 
-void CCameraMover::proc11(CErrorCode &errorCode, FVector &v, FMatrix &m) {
+void CCameraMover::updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation) {
 	if (_size > 0.0) {
-		v._x += m._row3._x * _size;
-		v._y += m._row3._y * _size;
-		v._z += m._row3._z * _size;
+		pos._x += orientation._row3._x * _size;
+		pos._y += orientation._row3._y * _size;
+		pos._z += orientation._row3._z * _size;
 
 		errorCode.set();
 	}
diff --git a/engines/titanic/star_control/camera_mover.h b/engines/titanic/star_control/camera_mover.h
index bd1f6b1..6577801 100644
--- a/engines/titanic/star_control/camera_mover.h
+++ b/engines/titanic/star_control/camera_mover.h
@@ -63,7 +63,12 @@ public:
 	virtual void moveTo(const FVector &srcV, const FVector &destV, const FMatrix &srcM) {}
 
 	virtual void proc10(const FVector &v1, const FVector &v2, const FVector &v3, const FMatrix &m) {}
-	virtual void proc11(CErrorCode &errorCode, FVector &v, FMatrix &m);
+
+	/**
+	 * Update the passed position and orientation matrix
+	 */
+	virtual void updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation);
+
 	virtual void setVector(CStarVector *sv);
 
 	/**
diff --git a/engines/titanic/star_control/star_camera.cpp b/engines/titanic/star_control/star_camera.cpp
index 5f69ebe..ede73b3 100644
--- a/engines/titanic/star_control/star_camera.cpp
+++ b/engines/titanic/star_control/star_camera.cpp
@@ -134,7 +134,7 @@ void CStarCamera::updatePosition(CErrorCode *errorCode) {
 
 	FVector priorPos = _viewport._position;
 	FVector newPos = _viewport._position;
-	_mover->proc11(*errorCode, newPos, *_newOrientation);
+	_mover->updatePosition(*errorCode, newPos, *_newOrientation);
 
 	if (newPos != priorPos) {
 		_viewport.setPosition(newPos);
diff --git a/engines/titanic/star_control/star_control_sub21.cpp b/engines/titanic/star_control/star_control_sub21.cpp
index 89ea3a1..ef6e8be 100644
--- a/engines/titanic/star_control/star_control_sub21.cpp
+++ b/engines/titanic/star_control/star_control_sub21.cpp
@@ -55,10 +55,10 @@ void CStarControlSub21::proc10(const FVector &v1, const FVector &v2, const FVect
 	incLockCount();
 }
 
-void CStarControlSub21::proc11(CErrorCode &errorCode, FVector &v, FMatrix &m) {
+void CStarControlSub21::updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation) {
 	if (_sub24.isActive()) {
 		decLockCount();
-		int val = _sub24.proc5(errorCode, v, m);
+		int val = _sub24.proc5(errorCode, pos, orientation);
 		if (val == 1)
 			incLockCount();
 		if (val == 2) {
@@ -67,9 +67,9 @@ void CStarControlSub21::proc11(CErrorCode &errorCode, FVector &v, FMatrix &m) {
 				_starVector->apply();
 		}
 	} else if (_size != 0.0) {
-		v._x += m._row3._x * _size;
-		v._y += m._row3._y * _size;
-		v._z += m._row3._z * _size;
+		pos._x += orientation._row3._x * _size;
+		pos._y += orientation._row3._y * _size;
+		pos._z += orientation._row3._z * _size;
 		errorCode.set();
 	}
 }
diff --git a/engines/titanic/star_control/star_control_sub21.h b/engines/titanic/star_control/star_control_sub21.h
index 4ebf823..bf1b050 100644
--- a/engines/titanic/star_control/star_control_sub21.h
+++ b/engines/titanic/star_control/star_control_sub21.h
@@ -37,7 +37,11 @@ public:
 
 	virtual void moveTo(const FVector &srcV, const FVector &destV, const FMatrix &srcM);
 	virtual void proc10(const FVector &v1, const FVector &v2, const FVector &v3, const FMatrix &m);
-	virtual void proc11(CErrorCode &errorCode, FVector &v, FMatrix &m);
+
+	/**
+	 * Update the passed position and orientation matrix
+	 */
+	virtual void updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation);
 };
 
 } // End of namespace Titanic
diff --git a/engines/titanic/star_control/star_control_sub22.cpp b/engines/titanic/star_control/star_control_sub22.cpp
index 8c3c7fa..3e88a32 100644
--- a/engines/titanic/star_control/star_control_sub22.cpp
+++ b/engines/titanic/star_control/star_control_sub22.cpp
@@ -37,10 +37,10 @@ void CStarControlSub22::proc8(FVector &v1, FVector &v2, FMatrix &m1, FMatrix &m2
 	incLockCount();
 }
 
-void CStarControlSub22::proc11(CErrorCode &errorCode, FVector &v, FMatrix &m) {
+void CStarControlSub22::updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation) {
 	if (_sub27.isActive()) {
 		decLockCount();
-		int val = _sub27.proc5(errorCode, v, m);
+		int val = _sub27.proc5(errorCode, pos, orientation);
 		if (val == 1)
 			incLockCount();
 		if (val == 2) {
@@ -49,9 +49,9 @@ void CStarControlSub22::proc11(CErrorCode &errorCode, FVector &v, FMatrix &m) {
 				_starVector->apply();
 		}
 	} else if (_size != 0.0) {
-		v._x += m._row3._x * _size;
-		v._y += m._row3._y * _size;
-		v._z += m._row3._z * _size;
+		pos._x += orientation._row3._x * _size;
+		pos._y += orientation._row3._y * _size;
+		pos._z += orientation._row3._z * _size;
 		errorCode.set();
 	}
 }
diff --git a/engines/titanic/star_control/star_control_sub22.h b/engines/titanic/star_control/star_control_sub22.h
index 5a597ef..dbb644b 100644
--- a/engines/titanic/star_control/star_control_sub22.h
+++ b/engines/titanic/star_control/star_control_sub22.h
@@ -36,7 +36,11 @@ public:
 	virtual ~CStarControlSub22() {}
 
 	virtual void proc8(FVector &v1, FVector &v2, FMatrix &m1, FMatrix &m2);
-	virtual void proc11(CErrorCode &errorCode, FVector &v, FMatrix &m);
+
+	/**
+	 * Update the passed position and orientation matrix
+	 */
+	virtual void updatePosition(CErrorCode &errorCode, FVector &pos, FMatrix &orientation);
 };
 
 } // End of namespace Titanic





More information about the Scummvm-git-logs mailing list