[Scummvm-git-logs] scummvm master -> e092ea2a693589a5ea77d5a8be035840dfa0c9ba

digitall noreply at scummvm.org
Mon May 22 22:02:54 UTC 2023


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:
e092ea2a69 WATCHMAKER: Add FIXME Comment Regarding Sequence Point Issue


Commit: e092ea2a693589a5ea77d5a8be035840dfa0c9ba
    https://github.com/scummvm/scummvm/commit/e092ea2a693589a5ea77d5a8be035840dfa0c9ba
Author: D G Turner (digitall at scummvm.org)
Date: 2023-05-22T23:02:04+01:00

Commit Message:
WATCHMAKER: Add FIXME Comment Regarding Sequence Point Issue

No functional change.

Changed paths:
    engines/watchmaker/walk/act.cpp


diff --git a/engines/watchmaker/walk/act.cpp b/engines/watchmaker/walk/act.cpp
index 76a56abfda2..81262843edc 100644
--- a/engines/watchmaker/walk/act.cpp
+++ b/engines/watchmaker/walk/act.cpp
@@ -493,6 +493,8 @@ void BuildStepList(int32 oc, uint8 dp, uint8 back) {
 
 	// arrotonda la fine
 	if ((w->NumSteps > 2)) {
+		// FIXME: The following code should be checked for correct intended logic as
+		//        it's previous form had sequence point issues.
 		angle = lastangle;
 		lastangle = w->WalkSteps[w->NumSteps - 2].Angle;
 		if (angle != lastangle) {




More information about the Scummvm-git-logs mailing list