- properly deal with 'gototitle'.

src/g_level.cpp

@@ -942,6 +942,13 @@ DIntermissionController* FLevelLocals::CreateIntermission()
 
 void RunIntermission(DIntermissionController* intermissionScreen, DObject* statusScreen, std::function<void(bool)> completionf)
 {
+			D_StartTitle();
+
+	if (!intermissionScreen && !statusScreen)
+	{
+		completionf(false);
+		return;
+	}
 	runner = CreateRunner(false);
 	GC::WriteBarrier(runner);
 	completion = std::move(completionf);
@@ -1010,9 +1017,9 @@ void G_DoCompleted (void)
 	bool endgame = intermissionScreen && intermissionScreen->mEndGame;
 	intermissionScreen = primaryLevel->CreateIntermission();
 	RunIntermission(intermissionScreen, statusScreen, [=](bool)
-					{
+	{
 		if (!endgame) primaryLevel->WorldDone();
-		
+		else if (!intermissionScreen) D_StartTitle();
 	});
 }
 

src/intermission/intermission.cpp

@@ -872,9 +872,7 @@ again:
 		}
 		else if (action->mClass == TITLE_ID)
 		{
-			Destroy();
+			continue;
-			D_StartTitle ();
-			return false;
 		}
 		else
 		{
@@ -1011,13 +1009,21 @@ DIntermissionController* F_StartIntermission(FIntermissionDescriptor *desc, bool
 DIntermissionController* F_StartIntermission(FName seq)
 {
 	FIntermissionDescriptor **pdesc = IntermissionDescriptors.CheckKey(seq);
-	if (pdesc == nullptr)
+	if (pdesc == nullptr || (*pdesc)->mActions.Size() == 0)
 	{
 		return nullptr;
 	}
 	else
 	{
-		return F_StartIntermission(*pdesc, false);
+		auto desc = *pdesc;
+		auto action = desc->mActions[0];
+		if (action->mClass == TITLE_ID)
+		{
+			// is handled elsewhere.
+			return nullptr;
+		}
+
+		return F_StartIntermission(desc, false);
 	}
 }
 

wadsrc/static/mapinfo/common.txt

@@ -412,9 +412,6 @@ Intermission Inter_Underwater
 	{
 		Background = "E2END", 0
 	}
-	GotoTitle
-	{
-	}
 }
 
 Intermission Inter_Demonscroll