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

digitall 547637+digitall at users.noreply.github.com
Sat Oct 19 19:51:42 CEST 2019


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:
b0ed255b39 CRUISE: Fix Missing Default Switch Cases


Commit: b0ed255b391b23ad0126e030f52a53783f4004c0
    https://github.com/scummvm/scummvm/commit/b0ed255b391b23ad0126e030f52a53783f4004c0
Author: D G Turner (digitall at scummvm.org)
Date: 2019-10-19T18:47:34+01:00

Commit Message:
CRUISE: Fix Missing Default Switch Cases

These are flagged by GCC if -Wswitch-default is enabled.

Changed paths:
    engines/cruise/background.cpp
    engines/cruise/dataLoader.cpp
    engines/cruise/linker.cpp
    engines/cruise/mainDraw.cpp
    engines/cruise/menu.cpp
    engines/cruise/script.cpp


diff --git a/engines/cruise/background.cpp b/engines/cruise/background.cpp
index 4cf52f6..4aa33cf 100644
--- a/engines/cruise/background.cpp
+++ b/engines/cruise/background.cpp
@@ -179,6 +179,7 @@ int loadBackground(const char *name, int idx) {
 
 		default:
 			assert(0);
+			break;
 		}
 
 		gfxModuleData_setPal256(palScreen[idx]);
@@ -199,6 +200,9 @@ int loadBackground(const char *name, int idx) {
 			memcpy(backgroundScreens[idx], ptr2, 320 * 200);
 			ptr2 += 320 * 200;
 			break;
+		default:
+			assert(0);
+			break;
 		}
 
 		loadMEN(&ptr2);
diff --git a/engines/cruise/dataLoader.cpp b/engines/cruise/dataLoader.cpp
index 2eff82b..d5b6541 100644
--- a/engines/cruise/dataLoader.cpp
+++ b/engines/cruise/dataLoader.cpp
@@ -128,6 +128,8 @@ void decodeGfxUnified(dataFileEntry *pCurrentFileEntry, int16 format) {
 
 		break;
 	}
+	default:
+		break;
 	}
 
 	MemFree(pCurrentFileEntry->subData.ptr);
diff --git a/engines/cruise/linker.cpp b/engines/cruise/linker.cpp
index 9786de7..68ed923 100644
--- a/engines/cruise/linker.cpp
+++ b/engines/cruise/linker.cpp
@@ -249,6 +249,8 @@ int updateScriptImport(int ovlIdx) {
 					ovlData->arrayMsgRelHeader[linkEntryIdx].obj2Number = pFoundExport->idx;
 					break;
 				}
+				default:
+					break;
 				}
 			}
 		}
diff --git a/engines/cruise/mainDraw.cpp b/engines/cruise/mainDraw.cpp
index 5777b84..015d0b0 100644
--- a/engines/cruise/mainDraw.cpp
+++ b/engines/cruise/mainDraw.cpp
@@ -1370,6 +1370,8 @@ int getValueFromObjectQuerry(objectParamsQuery *params, int idx) {
 		return params->state2;
 	case 7:
 		return params->nbState;
+	default:
+		break;
 	}
 
 	assert(0);
diff --git a/engines/cruise/menu.cpp b/engines/cruise/menu.cpp
index cf0b872..1abc7df 100644
--- a/engines/cruise/menu.cpp
+++ b/engines/cruise/menu.cpp
@@ -300,6 +300,8 @@ int playerMenu(int menuX, int menuY) {
 			break;
 		case 7: // exit
 			return 1;
+		default:
+			break;
 		}
 	}
 
diff --git a/engines/cruise/script.cpp b/engines/cruise/script.cpp
index aee3e8b..f4593a1 100644
--- a/engines/cruise/script.cpp
+++ b/engines/cruise/script.cpp
@@ -264,6 +264,8 @@ int32 opcodeType2() {
 
 	}
 	break;
+	default:
+		break;
 	}
 
 	return 0;
@@ -313,6 +315,9 @@ int32 opcodeType4() {		// test
 		if (var2 >= var1)
 			boolVar = 1;
 		break;
+
+	default:
+		break;
 	}
 
 	pushVar(boolVar);
@@ -398,6 +403,9 @@ int32 opcodeType5() {
 	case 7:
 		currentScriptPtr->scriptOffset = newSi;	//always
 		break;
+
+	default:
+		break;
 	}
 
 	return (0);
@@ -441,6 +449,8 @@ int32 opcodeType3()	{	// math
 		pushVar(pop2 & pop1);
 		return (0);
 
+	default:
+		break;
 	}
 
 	return 0;





More information about the Scummvm-git-logs mailing list