- fixed: A new episode definition for a map that already had one defined did not replace the old definition.

- fixed: Skipping the skill menu did not work because it checked the number of episodes, not skills.


SVN r3017 (trunk)

src/g_mapinfo.cpp

@@ -1626,7 +1626,13 @@ void FMapInfoParser::ParseEpisodeInfo ()
 	}
 	else
 	{
-		FEpisode *epi = &AllEpisodes[AllEpisodes.Reserve(1)];
+		// Only allocate a new entry if this doesn't replace an existing episode.
+		if (i >= AllEpisodes.Size())
+		{
+			i = AllEpisodes.Reserve(1);
+		}
+
+		FEpisode *epi = &AllEpisodes[i];
 
 		epi->mEpisodeMap = map;
 		epi->mEpisodeName = name;

src/menu/menudef.cpp

@@ -1327,7 +1327,7 @@ void M_StartupSkillMenu(FGameStartup *gs)
 			}
 			if (AllEpisodes[gs->Episode].mNoSkill || AllSkills.Size() == 1)
 			{
-				ld->mAutoselect = firstitem + MIN(2u, AllEpisodes.Size()-1);
+				ld->mAutoselect = firstitem + MIN(2u, AllSkills.Size()-1);
 			}
 			else
 			{