Added missing return values in VM calls

These are not supported by the JIT and must always be passed.

Author: Boondorl

src/common/cutscenes/screenjob.cpp

@@ -270,8 +270,10 @@ void ScreenJobDraw()
 		ScaleOverrider ovr(twod);
 		IFVIRTUALPTRNAME(cutscene.runner, NAME_ScreenJobRunner, RunFrame)
 		{
+			int ret = 0;
 			VMValue parm[] = { cutscene.runner, smoothratio };
-			VMCall(func, parm, 2, nullptr, 0);
+			VMReturn rets[] = { &ret };
+			VMCall(func, parm, 2, rets, 1);
 		}
 	}
 }

src/g_statusbar/sbarinfo_commands.cpp

@@ -1441,10 +1441,10 @@ class CommandDrawNumber : public CommandDrawString
 					static VMFunction *func = nullptr;
 					if (func == nullptr) PClass::FindFunction(&func, NAME_PlayerPawn, "GetEffectTicsForItem");
 					VMValue params[] = { statusBar->CPlayer->mo, inventoryItem };
-					int retv;
-					VMReturn ret(&retv);
-					VMCall(func, params, 2, &ret, 1);
-					num = retv < 0? 0 :  retv / TICRATE + 1;
+					int ret1 = 0, ret2 = 0;
+					VMReturn rets[] = { &ret1, &ret2 };
+					VMCall(func, params, 2, rets, 2);
+					num = ret1 < 0? 0 :  ret1 / TICRATE + 1;
 					break;
 				}
 				case INVENTORY:

src/intermission/intermission.cpp

@@ -325,8 +325,10 @@ void DIntermissionScreenCutscene::Drawer ()
 	ScaleOverrider ovr(twod);
 	IFVIRTUALPTRNAME(mScreenJobRunner, NAME_ScreenJobRunner, RunFrame)
 	{
+		int res = 0;
 		VMValue parm[] = { mScreenJobRunner, I_GetTimeFrac() };
-		VMCall(func, parm, 2, nullptr, 0);
+		VMReturn ret[] = { &res };
+		VMCall(func, parm, 2, ret, 1);
 	}
 }
 

src/p_conversation.cpp

@@ -218,8 +218,10 @@ static void TakeStrifeItem (player_t *player, PClassActor *itemtype, int amount)
 
 	IFVM(Actor, TakeInventory)
 	{
+		int taken = false;
 		VMValue params[] = { player->mo, itemtype, amount, false, false };
-		VMCall(func, params, 5, nullptr, 0);
+		VMReturn rets[] = { &taken };
+		VMCall(func, params, 5, rets, 1);
 	}
 }
 

src/playsim/fragglescript/t_func.cpp

@@ -2532,8 +2532,10 @@ void FParser::SF_PlayerWeapon()
 
 						IFVM(PlayerPawn, PickNewWeapon)
 						{
+							AActor* weap = nullptr;
 							VMValue param[] = { Level->Players[playernum]->mo, (void*)nullptr };
-							VMCall(func, param, 2, nullptr, 0);
+							VMReturn rets[] = { (void**)&weap };
+							VMCall(func, param, 2, rets, 1);
 						}
 					}
 				}

src/playsim/p_interaction.cpp

@@ -345,7 +345,10 @@ void AActor::Die (AActor *source, AActor *inflictor, int dmgflags, FName MeansOf
 				}
 			}
 
-			VMCall(func, params, 1, nullptr, 0);
+			AActor* unused = nullptr;
+			int unused2 = 0, unused3 = 0;
+			VMReturn ret[] = { (void**)&unused, &unused2, &unused3 };
+			VMCall(func, params, 1, ret, 3);
 
 			// Kill the dummy Actor if it didn't unmorph, otherwise checking the morph flags. Player pawns need
 			// to stay, otherwise they won't respawn correctly.

src/playsim/p_map.cpp

@@ -215,7 +215,7 @@ bool P_CanCrossLine(AActor *mo, line_t *line, DVector3 next)
 		assert(VIndex != ~0u);
 	}
 
-	VMValue params[] = { mo, line, next.X, next.Y, next.Z, false };
+	VMValue params[] = { mo, line, next.X, next.Y, next.Z };
 	VMReturn ret;
 	int retval;
 	ret.IntAt(&retval);

src/playsim/p_teleport.cpp

@@ -249,7 +249,7 @@ bool P_Teleport (AActor *thing, DVector3 pos, DAngle angle, int flags)
 		IFVIRTUALPTR(thing, AActor, PostTeleport)
 		{
 			VMValue params[] = { thing, pos.X, pos.Y, pos.Z, angle.Degrees(), flags };
-			VMCall(func, params, countof(params), nullptr, 1);
+			VMCall(func, params, countof(params), nullptr, 0);
 		}
 	}
 	return true;