Fix p1 4104 and individual demos not timing with tick 0

ConsoleApp/src/DemoArgProcessing/DemoParserSubCommand.cs

@@ -63,11 +63,7 @@ private void EnableTimeOptionDefault(DemoParsingSetupInfo setupInfo) {
 				// Due to the portal community's love for hundreds of saveloads, the time option will time w/ first tick by default
 				// for p1 and l4d (because hindsight or something). This is a hack for now and will hopefully by improved with a
 				// more advanced timing system in the utopian future.
-				((OptTime)option).SetForceFirstTickTiming(new[] {
-					SourceGame.L4D1_1005, SourceGame.L4D1_1040, SourceGame.L4D2_2000, SourceGame.L4D2_2011,
-					SourceGame.L4D2_2012, SourceGame.L4D2_2027, SourceGame.L4D2_2042, SourceGame.L4D2_2091, SourceGame.L4D2_2147,
-					SourceGame.L4D2_2203, SourceGame.PORTAL_1_3420, SourceGame.PORTAL_1_5135, SourceGame.PORTAL_1_1910503
-				});
+				((OptTime)option).SetForceFirstTickTiming((SourceGame game) => game.IsPortal1() || game.IsLeft4Dead());
 			} else {
 				throw new ArgProcessProgrammerException("listdemo option not passed to demo sub-command.");
 			}

ConsoleApp/src/DemoArgProcessing/Options/OptTime.cs

@@ -20,9 +20,6 @@ public class OptTime : DemoOption<OptTime.TimeFlags> {
 
 		private const int FmtIdt = -25;
 
-		// a hack to allow only p1 & l4d to time w/ first tick
-		private ISet<SourceGame> _forceFirstTickTiming = null!;
-
 
 		[Flags]
 		public enum TimeFlags {
@@ -51,7 +48,6 @@ public OptTime() : base(
 		public override void Reset() {
 			base.Reset();
 			_sdt = null!;
-			_forceFirstTickTiming = new HashSet<SourceGame>();
 		}
 
 
@@ -71,7 +67,7 @@ protected override void Process(DemoParsingInfo infoObj, TimeFlags arg, bool isD
 					WriteAdjustedTime(
 						infoObj.CurrentDemo,
 						Console.Out,
-						(arg & TimeFlags.TimeFirstTick) != 0 || _forceFirstTickTiming.Contains(infoObj.CurrentDemo.DemoInfo.Game)
+						(arg & TimeFlags.TimeFirstTick) != 0 || _sdt.TimeFirstTick
 					);
 					Console.WriteLine();
 				}
@@ -207,15 +203,15 @@ public static void WriteAdjustedTime(SourceDemo demo, TextWriter tw, bool timeFi
 			tw.WriteLine($"{"Measured ticks ",FmtIdt}: {demo.TickCount(tfs)}");
 
 			// hack before we implement proper timing strategies
-			if (demo.DemoInfo.IsLeft4Dead1() && demo.SequenceType == TimingAdjustment.SequenceType.SingleDemo && demo.StartAdjustmentTick.HasValue) {
+			if (demo.DemoInfo.Game.IsLeft4Dead1() && demo.SequenceType == TimingAdjustment.SequenceType.SingleDemo && demo.StartAdjustmentTick.HasValue) {
 				// all maps except finales will have a map_transition event at the end if the map was completed
 				(var gameEvent, int? endTick) = demo.FilterForMessage<SvcGameEvent>().FirstOrDefault(ev => ev.message.EventDescription!.Name == "map_transition");
 
 				// assume it's a finale if we don't find a map_transition event and copy the end adjustment tick
 				if (gameEvent == null)
 					endTick = demo.EndAdjustmentTick;
 				if (endTick.HasValue) {
-					int mapTime = endTick.Value - demo.StartAdjustmentTick.Value + 1;
+					int mapTime = endTick!.Value - demo.StartAdjustmentTick.Value + 1;
 					tw.WriteLine($"{"Individual Map time ",FmtIdt}: {Utils.FormatTime(mapTime * tickInterval)}");
 					tw.Write($"{"Individual Map ticks ",FmtIdt}: {mapTime}");
 					tw.WriteLine($" ({demo.StartAdjustmentTick}-{endTick})");
@@ -229,7 +225,7 @@ public static void WriteAdjustedTime(SourceDemo demo, TextWriter tw, bool timeFi
 			}
 
 			// another hack until we time things better :)
-			if (demo.DemoInfo.IsPortal2()) {
+			if (demo.DemoInfo.Game.IsPortal2()) {
 				ScoreboardTempUpdate? lastScoreBoard =
 					demo.FilterForUserMessage<ScoreboardTempUpdate>()
 						.Select(tuple => tuple.userMessage)
@@ -249,10 +245,8 @@ public static void WriteAdjustedTime(SourceDemo demo, TextWriter tw, bool timeFi
 
 
 		// a hack to allow only p1 & l4d to time w/ first tick
-		internal void SetForceFirstTickTiming(IEnumerable<SourceGame> games) {
-			var gameSet = games.ToImmutableHashSet();
-			_forceFirstTickTiming = gameSet;
-			_sdt.SetForceFirstTickTiming(gameSet);
+		internal void SetForceFirstTickTiming(Predicate<SourceGame> pred) {
+			_sdt.SetForceFirstTickTiming(pred);
 		}
 	}
 
@@ -286,14 +280,14 @@ public SimpleDemoInfo(SourceDemo demo) {
 
 		// a hack to allow only p1 & l4d to time w/ first tick
 		private SourceDemo _curDemo = null!;
-		private ISet<SourceGame> _forceFirstTickTiming;
-		public bool TimeFirstTick => _timeFirstTick || _forceFirstTickTiming.Contains(_curDemo.DemoInfo.Game);
+		private Predicate<SourceGame> _forceFirstTickTiming;
+		public bool TimeFirstTick => _timeFirstTick || _forceFirstTickTiming(_curDemo.DemoInfo.Game);
 
 
 		public SimpleDemoTimer(bool timeFirstTick) {
 			_timeFirstTick = timeFirstTick;
 			ValidFlags = Flags.TotalTimeValid | Flags.AdjustedTimeValid;
-			_forceFirstTickTiming = new HashSet<SourceGame>();
+			_forceFirstTickTiming = (SourceGame) => false;
 			_simpleDemoDifferences = new HashSet<string>();
 		}
 
@@ -304,7 +298,7 @@ public SimpleDemoTimer(IEnumerable<SourceDemo> demos, bool timeFirstTick) : this
 		}
 
 
-		internal void SetForceFirstTickTiming(IEnumerable<SourceGame> games) => _forceFirstTickTiming = games.ToImmutableHashSet();
+		internal void SetForceFirstTickTiming(Predicate<SourceGame> pred) => _forceFirstTickTiming = pred;
 
 
 		/// <summary>

DemoParser/src/Parser/Components/Messages/SvcCreateStringTable.cs

@@ -24,7 +24,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 			TableName = bsr.ReadNullTerminatedString();
 			MaxEntries = (short)bsr.ReadUShort();
 			NumEntries = (int)bsr.ReadUInt(ParserUtils.HighestBitIndex((uint)MaxEntries) + 1);
-			uint dataLen = bsr.ReadUInt(DemoInfo.IsLeft4Dead2() ? 21 : 20);
+			uint dataLen = bsr.ReadUInt(DemoInfo.Game.IsLeft4Dead2() ? 21 : 20);
 			UserDataFixedSize = bsr.ReadBool();
 			UserDataSize = (int)(UserDataFixedSize ? bsr.ReadUInt(12) : 0);
 			UserDataSizeBits = (int)(UserDataFixedSize ? bsr.ReadUInt(4) : 0);

DemoParser/src/Parser/Components/Messages/SvcPacketEntities.cs

@@ -140,7 +140,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 				Updates.Add(update);
 			};
 			if (IsDelta) {
-				if (DemoInfo.IsLeft4Dead2() && DemoInfo.Game >= SourceGame.L4D2_2091) {
+				if (DemoInfo.Game.IsLeft4Dead2() && DemoInfo.Game >= SourceGame.L4D2_2091) {
 					idx = -1;
 					uint deletionCount = entBsr.ReadUBitInt();
 					for (int i = 0; i < deletionCount; i++) {

DemoParser/src/Parser/Components/Messages/SvcPrefetch.cs

@@ -15,7 +15,7 @@ public SvcPrefetch(SourceDemo? demoRef, byte value) : base(demoRef, value) {}
 
 
 		protected override void Parse(ref BitStreamReader bsr) {
-			var soundIndexBits = DemoInfo.IsLeft4Dead2() ? DemoInfo.Game >= SourceGame.L4D2_2091 ? 15 : 14 : 13;
+			var soundIndexBits = DemoInfo.Game.IsLeft4Dead2() ? DemoInfo.Game >= SourceGame.L4D2_2091 ? 15 : 14 : 13;
 			SoundIndex = (int)bsr.ReadUInt(soundIndexBits);
 
 			var mgr = GameState.StringTablesManager;

DemoParser/src/Parser/Components/Messages/SvcServerInfo.cs

@@ -42,7 +42,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 			ServerCount = bsr.ReadUInt();
 			IsHltv = bsr.ReadBool();
 			IsDedicated = bsr.ReadBool();
-			if (DemoInfo.IsLeft4Dead() && DemoInfo.Game >= SourceGame.L4D2_2147)
+			if (DemoInfo.Game.IsLeft4Dead() && DemoInfo.Game >= SourceGame.L4D2_2147)
 				RestrictWorkshopAddons = bsr.ReadBool();
 			ClientCrc = bsr.ReadSInt();
 			if (DemoInfo.NewDemoProtocol)
@@ -61,7 +61,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 			MapName = bsr.ReadNullTerminatedString();
 			SkyName = bsr.ReadNullTerminatedString();
 			HostName = bsr.ReadNullTerminatedString();
-			if (DemoInfo.IsLeft4Dead() && DemoInfo.Game >= SourceGame.L4D2_2147)
+			if (DemoInfo.Game.IsLeft4Dead() && DemoInfo.Game >= SourceGame.L4D2_2147)
 			{
 				MissionName = bsr.ReadNullTerminatedString();
 				MutationName = bsr.ReadNullTerminatedString();

DemoParser/src/Parser/Components/Messages/SvcTempEntities.cs

@@ -17,7 +17,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 			EntryCount = bsr.ReadByte();
 			uint bitLen = DemoInfo.Game is SourceGame.PORTAL_1_1910503 or SourceGame.TF2
 				? bsr.ReadVarUInt32()
-				: bsr.ReadUInt(DemoInfo.IsLeft4Dead2() ? 18 : 17);
+				: bsr.ReadUInt(DemoInfo.Game.IsLeft4Dead2() ? 18 : 17);
 			_data = bsr.ForkAndSkip((int)bitLen);
 
 			// todo

DemoParser/src/Parser/Components/Messages/SvcVoiceData.cs

@@ -19,7 +19,7 @@ protected override void Parse(ref BitStreamReader bsr) {
 		FromClient = bsr.ReadByte();
 		Proximity = bsr.ReadByte() != 0;
 		BitLen = bsr.ReadUShort();
-		if (DemoRef.DemoInfo.IsLeft4Dead()) {
+		if (DemoRef.DemoInfo.Game.IsLeft4Dead()) {
 			Unknown = new bool[4];
 			for (int i = 0; i < 4; i++)
 				Unknown[i] = bsr.ReadBool();

DemoParser/src/Parser/Components/Packets/StringTableEntryTypes/PlayerInfo.cs

@@ -31,7 +31,7 @@ public PlayerInfo(SourceDemo? demoRef, int? decompressedIndex = null) : base(dem
 
 		// we're reading a player_info_t, so take alignment into account
 		protected override void Parse(ref BitStreamReader bsr) {
-            if (DemoInfo.NewDemoProtocol && !DemoInfo.IsLeft4Dead1())
+            if (DemoInfo.NewDemoProtocol && !DemoInfo.Game.IsLeft4Dead1())
                 SteamId = (CSteamId)bsr.ReadULong();
 			Name = bsr.ReadStringOfLength(MaxPlayerNameLength);
 			UserId = bsr.ReadSInt();