[Scummvm-cvs-logs] scummvm master -> c4b3f113d3760d3c370259fd938465dab7a371c8

eriktorbjorn eriktorbjorn at telia.com
Sat Jun 1 22:20:24 CEST 2013


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:
c4b3f113d3 SAGA: Add missing "break"s


Commit: c4b3f113d3760d3c370259fd938465dab7a371c8
    https://github.com/scummvm/scummvm/commit/c4b3f113d3760d3c370259fd938465dab7a371c8
Author: Torbjörn Andersson (eriktorbjorn at users.sourceforge.net)
Date: 2013-06-01T13:16:32-07:00

Commit Message:
SAGA: Add missing "break"s

It seems pretty clear to me that these fallthroughs were not
intentional, seeing how one of them could cause ITE to fall through
to IHNM-specific code.

CID 1003744, 1003745

Changed paths:
    engines/saga/events.cpp



diff --git a/engines/saga/events.cpp b/engines/saga/events.cpp
index d98cef0..c5842f4 100644
--- a/engines/saga/events.cpp
+++ b/engines/saga/events.cpp
@@ -533,6 +533,7 @@ int Events::handleOneShot(Event *event) {
 		default:
 			break;
 		}
+		break;
 #ifdef ENABLE_IHNM
 	case kCutawayEvent:
 		switch (event->op) {
@@ -545,6 +546,7 @@ int Events::handleOneShot(Event *event) {
 		default:
 			break;
 		}
+		break;
 #endif
 	case kActorEvent:
 		switch (event->op) {
@@ -554,6 +556,7 @@ int Events::handleOneShot(Event *event) {
 		default:
 			break;
 		}
+		break;
 	default:
 		break;
 	}






More information about the Scummvm-git-logs mailing list