From 606ba09b5fb08b6d0408fcdd3190dfd518b9f79a Mon Sep 17 00:00:00 2001
From: Andreas Pardeike <andreas@pardeike.net>
Date: Fri, 29 Dec 2023 15:53:30 +0100
Subject: [PATCH] code cleanup, net8.0, optional ret code for methods ending in
 throw

---
 Harmony/Documentation/Documentation.csproj    |  2 +-
 Harmony/Harmony.csproj                        |  5 +-
 Harmony/Internal/AccessCache.cs               | 18 +++----
 Harmony/Internal/CodeTranspiler.cs            |  4 +-
 Harmony/Internal/Emitter.cs                   |  2 +-
 Harmony/Internal/HarmonySharedState.cs        |  2 +-
 Harmony/Internal/ILInstruction.cs             |  4 +-
 Harmony/Internal/MethodCopier.cs              |  6 +--
 Harmony/Internal/MethodPatcher.cs             |  3 +-
 Harmony/Internal/PatchModels.cs               | 10 ++--
 Harmony/Internal/PatchSorter.cs               |  8 +--
 Harmony/Internal/PatchTools.cs                |  2 +-
 Harmony/Public/CodeInstruction.cs             | 10 ++--
 Harmony/Public/InlineSignature.cs             |  2 +-
 Harmony/Public/PatchClassProcessor.cs         | 22 ++++----
 Harmony/Tools/AccessTools.cs                  | 31 ++++++-----
 Harmony/Tools/AccessToolsExtensions.cs        |  4 +-
 Harmony/Tools/CodeMatch.cs                    |  8 +--
 Harmony/Tools/CodeMatcher.cs                  |  6 +--
 Harmony/Tools/Extensions.cs                   | 54 +++++++++----------
 Harmony/Tools/FileLog.cs                      |  4 +-
 HarmonyTests/HarmonyTests.csproj              |  4 +-
 HarmonyTests/IL/DynamicArgumentPatches.cs     |  4 +-
 .../Patching/Assets/CombinedPatchClass.cs     | 16 +++++-
 HarmonyTests/Patching/Assets/PatchClasses.cs  | 18 +++----
 HarmonyTests/Patching/Assets/Specials.cs      |  8 +--
 HarmonyTests/Patching/FinalizerPatches.cs     |  2 +-
 HarmonyTests/Patching/GenericsPatches.cs      |  6 +--
 HarmonyTests/Patching/Transpiling.cs          |  2 +-
 HarmonyTests/Tools/Assets/AccessToolsClass.cs |  2 +-
 HarmonyTests/Tools/TestFieldRefAccess.cs      |  2 +-
 HarmonyTests/Traverse/TestTraverse_Basics.cs  |  2 +-
 TestLibrary/TestLibrary.csproj                |  2 +-
 TestLibrary/TestPatch.cs                      |  2 +-
 azure-pipelines.yml                           |  8 +--
 testEnvironments.json                         | 10 ++++
 36 files changed, 163 insertions(+), 132 deletions(-)
 create mode 100644 testEnvironments.json

diff --git a/Harmony/Documentation/Documentation.csproj b/Harmony/Documentation/Documentation.csproj
index d1b1ac8b..584cff2c 100644
--- a/Harmony/Documentation/Documentation.csproj
+++ b/Harmony/Documentation/Documentation.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
-		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0</TargetFrameworks>
+		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0;net8.0</TargetFrameworks>
 		<LangVersion>preview</LangVersion>
 		<IsPackable>false</IsPackable>
 		<IntermediateOutputPath>obj</IntermediateOutputPath>
diff --git a/Harmony/Harmony.csproj b/Harmony/Harmony.csproj
index 35e9b537..0daf6891 100644
--- a/Harmony/Harmony.csproj
+++ b/Harmony/Harmony.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
-		<TargetFrameworks>netstandard2.0;net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0</TargetFrameworks>
+		<TargetFrameworks>netstandard2.0;net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0;net8.0</TargetFrameworks>
 		<GeneratePackageOnBuild>true</GeneratePackageOnBuild>
 		<Product>Harmony</Product>
 		<Company>Andreas Pardeike</Company>
@@ -35,7 +35,7 @@
 
 	<!-- In .NET 5.0 Binary Formatters are off by default.  Support is added for .NET 5.0+ for fallback JSON Serialization -->
 	<Choose>
-		<When Condition="'$(TargetFramework)'=='net5.0' Or '$(TargetFramework)'=='net6.0' Or '$(TargetFramework)'=='net7.0'">
+		<When Condition="'$(TargetFramework)'=='net5.0' Or '$(TargetFramework)'=='net6.0' Or '$(TargetFramework)'=='net7.0' Or '$(TargetFramework)'=='net8.0'">
 			<PropertyGroup>
 				<IsNET5OrGreater>true</IsNET5OrGreater>
 			</PropertyGroup>
@@ -122,6 +122,7 @@
 		<Compile Remove="**\*.net5.cs" />
 		<Compile Remove="**\*.net6.cs" />
 		<Compile Remove="**\*.net7.cs" />
+		<Compile Remove="**\*.net8.cs" />
 	</ItemGroup>
 
 	<Target Name="RemoveOldNuGetPackages" BeforeTargets="PreBuildEvent">
diff --git a/Harmony/Internal/AccessCache.cs b/Harmony/Internal/AccessCache.cs
index 89d922c7..ed8c4046 100644
--- a/Harmony/Internal/AccessCache.cs
+++ b/Harmony/Internal/AccessCache.cs
@@ -21,13 +21,13 @@ internal enum MemberType
 			{ MemberType.Static, BasicFlags | BindingFlags.Static }
 		};
 
-		readonly Dictionary<Type, Dictionary<string, FieldInfo>> declaredFields = new();
-		readonly Dictionary<Type, Dictionary<string, PropertyInfo>> declaredProperties = new();
-		readonly Dictionary<Type, Dictionary<string, Dictionary<int, MethodBase>>> declaredMethods = new();
+		readonly Dictionary<Type, Dictionary<string, FieldInfo>> declaredFields = [];
+		readonly Dictionary<Type, Dictionary<string, PropertyInfo>> declaredProperties = [];
+		readonly Dictionary<Type, Dictionary<string, Dictionary<int, MethodBase>>> declaredMethods = [];
 
-		readonly Dictionary<Type, Dictionary<string, FieldInfo>> inheritedFields = new();
-		readonly Dictionary<Type, Dictionary<string, PropertyInfo>> inheritedProperties = new();
-		readonly Dictionary<Type, Dictionary<string, Dictionary<int, MethodBase>>> inheritedMethods = new();
+		readonly Dictionary<Type, Dictionary<string, FieldInfo>> inheritedFields = [];
+		readonly Dictionary<Type, Dictionary<string, PropertyInfo>> inheritedProperties = [];
+		readonly Dictionary<Type, Dictionary<string, Dictionary<int, MethodBase>>> inheritedMethods = [];
 
 		static T Get<T>(Dictionary<Type, Dictionary<string, T>> dict, Type type, string name, Func<T> fetcher)
 		{
@@ -35,7 +35,7 @@ static T Get<T>(Dictionary<Type, Dictionary<string, T>> dict, Type type, string
 			{
 				if (dict.TryGetValue(type, out var valuesByName) is false)
 				{
-					valuesByName = new Dictionary<string, T>();
+					valuesByName = [];
 					dict[type] = valuesByName;
 				}
 				if (valuesByName.TryGetValue(name, out var value) is false)
@@ -53,12 +53,12 @@ static T Get<T>(Dictionary<Type, Dictionary<string, Dictionary<int, T>>> dict, T
 			{
 				if (dict.TryGetValue(type, out var valuesByName) is false)
 				{
-					valuesByName = new Dictionary<string, Dictionary<int, T>>();
+					valuesByName = [];
 					dict[type] = valuesByName;
 				}
 				if (valuesByName.TryGetValue(name, out var valuesByArgument) is false)
 				{
-					valuesByArgument = new Dictionary<int, T>();
+					valuesByArgument = [];
 					valuesByName[name] = valuesByArgument;
 				}
 				var argumentsHash = AccessTools.CombinedHashCode(arguments);
diff --git a/Harmony/Internal/CodeTranspiler.cs b/Harmony/Internal/CodeTranspiler.cs
index 14adb97e..31020c87 100644
--- a/Harmony/Internal/CodeTranspiler.cs
+++ b/Harmony/Internal/CodeTranspiler.cs
@@ -10,7 +10,7 @@ namespace HarmonyLib
 	internal class CodeTranspiler
 	{
 		readonly IEnumerable<CodeInstruction> codeInstructions;
-		readonly List<MethodInfo> transpilers = new();
+		readonly List<MethodInfo> transpilers = [];
 
 		internal CodeTranspiler(List<ILInstruction> ilInstructions)
 		{
@@ -156,7 +156,7 @@ internal static IEnumerable ConvertInstructionsAndUnassignedValues(Type type, IE
 			var listType = enumerableAssembly.GetType(genericListTypeWithElement.FullName);
 			var list = Activator.CreateInstance(listType);
 			var listAdd = list.GetType().GetMethod("Add");
-			unassignedValues = new Dictionary<object, Dictionary<string, object>>();
+			unassignedValues = [];
 			foreach (var op in enumerable)
 			{
 				var elementTo = ConvertInstruction(elementType, op, out var unassigned);
diff --git a/Harmony/Internal/Emitter.cs b/Harmony/Internal/Emitter.cs
index 4221e2c9..a7357cd5 100644
--- a/Harmony/Internal/Emitter.cs
+++ b/Harmony/Internal/Emitter.cs
@@ -19,7 +19,7 @@ public override string ToString()
 	internal class Emitter
 	{
 		readonly CecilILGenerator il;
-		readonly Dictionary<int, CodeInstruction> instructions = new();
+		readonly Dictionary<int, CodeInstruction> instructions = [];
 		readonly bool debug;
 
 		internal Emitter(ILGenerator il, bool debug)
diff --git a/Harmony/Internal/HarmonySharedState.cs b/Harmony/Internal/HarmonySharedState.cs
index d632e764..4b145cd5 100644
--- a/Harmony/Internal/HarmonySharedState.cs
+++ b/Harmony/Internal/HarmonySharedState.cs
@@ -65,7 +65,7 @@ static HarmonySharedState()
 			state = (Dictionary<MethodBase, byte[]>)stateField.GetValue(null);
 
 			// copy 'originals' over to our fields
-			originals = new Dictionary<MethodInfo, MethodBase>();
+			originals = [];
 			if (originalsField != null) // may not exist in older versions
 				originals = (Dictionary<MethodInfo, MethodBase>)originalsField.GetValue(null);
 		}
diff --git a/Harmony/Internal/ILInstruction.cs b/Harmony/Internal/ILInstruction.cs
index 454aed2e..d84d9408 100644
--- a/Harmony/Internal/ILInstruction.cs
+++ b/Harmony/Internal/ILInstruction.cs
@@ -11,8 +11,8 @@ internal class ILInstruction
 		internal object operand;
 		internal object argument;
 
-		internal List<Label> labels = new();
-		internal List<ExceptionBlock> blocks = new();
+		internal List<Label> labels = [];
+		internal List<ExceptionBlock> blocks = [];
 
 		internal ILInstruction(OpCode opcode, object operand = null)
 		{
diff --git a/Harmony/Internal/MethodCopier.cs b/Harmony/Internal/MethodCopier.cs
index 1e8c3cdf..39e87519 100644
--- a/Harmony/Internal/MethodCopier.cs
+++ b/Harmony/Internal/MethodCopier.cs
@@ -14,7 +14,7 @@ namespace HarmonyLib
 	internal class MethodCopier
 	{
 		readonly MethodBodyReader reader;
-		readonly List<MethodInfo> transpilers = new();
+		readonly List<MethodInfo> transpilers = [];
 
 		internal MethodCopier(MethodBase fromMethod, ILGenerator toILGenerator, LocalBuilder[] existingVariables = null)
 		{
@@ -98,7 +98,7 @@ internal MethodBodyReader(MethodBase method, ILGenerator generator)
 			if ((body?.GetILAsByteArray()?.Length ?? 0) == 0)
 			{
 				ilBytes = new ByteBuffer(new byte[0]);
-				ilInstructions = new List<ILInstruction>();
+				ilInstructions = [];
 			}
 			else
 			{
@@ -127,7 +127,7 @@ internal MethodBodyReader(MethodBase method, ILGenerator generator)
 				this_parameter = new ThisParameter(method);
 			parameters = method.GetParameters();
 
-			localVariables = body?.LocalVariables?.ToList() ?? new List<LocalVariableInfo>();
+			localVariables = body?.LocalVariables?.ToList() ?? [];
 			exceptions = body?.ExceptionHandlingClauses ?? new List<ExceptionHandlingClause>();
 		}
 
diff --git a/Harmony/Internal/MethodPatcher.cs b/Harmony/Internal/MethodPatcher.cs
index 2507f4d8..ab367ce9 100644
--- a/Harmony/Internal/MethodPatcher.cs
+++ b/Harmony/Internal/MethodPatcher.cs
@@ -206,7 +206,8 @@ internal MethodInfo CreateReplacement(out Dictionary<int, CodeInstruction> final
 					emitter.Emit(OpCodes.Ldloc, resultVariable);
 			}
 
-			emitter.Emit(OpCodes.Ret);
+			if (endsInThrow == false || hasFinalizers) // methods ending in throw cannot have more code after the throw
+				emitter.Emit(OpCodes.Ret);
 
 			finalInstructions = emitter.GetInstructions();
 
diff --git a/Harmony/Internal/PatchModels.cs b/Harmony/Internal/PatchModels.cs
index c4e65550..8a430cd4 100644
--- a/Harmony/Internal/PatchModels.cs
+++ b/Harmony/Internal/PatchModels.cs
@@ -14,10 +14,10 @@ internal class Job
 		{
 			internal MethodBase original;
 			internal T replacement;
-			internal List<HarmonyMethod> prefixes = new();
-			internal List<HarmonyMethod> postfixes = new();
-			internal List<HarmonyMethod> transpilers = new();
-			internal List<HarmonyMethod> finalizers = new();
+			internal List<HarmonyMethod> prefixes = [];
+			internal List<HarmonyMethod> postfixes = [];
+			internal List<HarmonyMethod> transpilers = [];
+			internal List<HarmonyMethod> finalizers = [];
 
 			internal void AddPatch(AttributePatch patch)
 			{
@@ -39,7 +39,7 @@ internal void AddPatch(AttributePatch patch)
 			}
 		}
 
-		internal Dictionary<MethodBase, Job> state = new();
+		internal Dictionary<MethodBase, Job> state = [];
 
 		internal Job GetJob(MethodBase method)
 		{
diff --git a/Harmony/Internal/PatchSorter.cs b/Harmony/Internal/PatchSorter.cs
index f9ced1d9..6fd226d0 100644
--- a/Harmony/Internal/PatchSorter.cs
+++ b/Harmony/Internal/PatchSorter.cs
@@ -44,8 +44,8 @@ internal List<MethodInfo> Sort(MethodBase original)
 			if (sortedPatchArray is not null) return sortedPatchArray.Select(x => x.GetMethod(original)).ToList();
 
 			// Initialize internal structures used for sorting.
-			handledPatches = new HashSet<PatchSortingWrapper>();
-			waitingList = new List<PatchSortingWrapper>();
+			handledPatches = [];
+			waitingList = [];
 			result = new List<PatchSortingWrapper>(patches.Count);
 			var queue = new Queue<PatchSortingWrapper>(patches);
 
@@ -165,8 +165,8 @@ class PatchSortingWrapper : IComparable
 			internal PatchSortingWrapper(Patch patch)
 			{
 				innerPatch = patch;
-				before = new HashSet<PatchSortingWrapper>();
-				after = new HashSet<PatchSortingWrapper>();
+				before = [];
+				after = [];
 			}
 
 			/// <summary>Determines how patches sort</summary>
diff --git a/Harmony/Internal/PatchTools.cs b/Harmony/Internal/PatchTools.cs
index 4ec1b09a..53e795d2 100644
--- a/Harmony/Internal/PatchTools.cs
+++ b/Harmony/Internal/PatchTools.cs
@@ -11,7 +11,7 @@ namespace HarmonyLib
 {
 	internal static class PatchTools
 	{
-		private static readonly Dictionary<MethodBase, ICoreDetour> detours = new();
+		private static readonly Dictionary<MethodBase, ICoreDetour> detours = [];
 
 		internal static void DetourMethod(MethodBase method, MethodBase replacement)
 		{
diff --git a/Harmony/Public/CodeInstruction.cs b/Harmony/Public/CodeInstruction.cs
index 6791235e..b26c4b2c 100644
--- a/Harmony/Public/CodeInstruction.cs
+++ b/Harmony/Public/CodeInstruction.cs
@@ -13,7 +13,7 @@ public class CodeInstruction
 	{
 		internal static class State
 		{
-			internal static readonly Dictionary<int, Delegate> closureCache = new();
+			internal static readonly Dictionary<int, Delegate> closureCache = [];
 		}
 
 		/// <summary>The opcode</summary>
@@ -26,11 +26,11 @@ internal static class State
 
 		/// <summary>All labels defined on this instruction</summary>
 		///
-		public List<Label> labels = new();
+		public List<Label> labels = [];
 
 		/// <summary>All exception block boundaries defined on this instruction</summary>
 		///
-		public List<ExceptionBlock> blocks = new();
+		public List<ExceptionBlock> blocks = [];
 
 		// Internal parameterless constructor that AccessTools.CreateInstance can use, ensuring that labels/blocks are initialized.
 		internal CodeInstruction()
@@ -67,8 +67,8 @@ public CodeInstruction Clone()
 		{
 			return new CodeInstruction(this)
 			{
-				labels = new List<Label>(),
-				blocks = new List<ExceptionBlock>()
+				labels = [],
+				blocks = []
 			};
 		}
 
diff --git a/Harmony/Public/InlineSignature.cs b/Harmony/Public/InlineSignature.cs
index 2493493e..f6f5a4ee 100644
--- a/Harmony/Public/InlineSignature.cs
+++ b/Harmony/Public/InlineSignature.cs
@@ -27,7 +27,7 @@ internal class InlineSignature : ICallSiteGenerator
 
 		/// <summary>The list of all parameter types or function pointer signatures received by the call site</summary>
 		///
-		public List<object> Parameters { get; set; } = new List<object>();
+		public List<object> Parameters { get; set; } = [];
 
 		/// <summary>The return type or function pointer signature returned by the call site</summary>
 		///
diff --git a/Harmony/Public/PatchClassProcessor.cs b/Harmony/Public/PatchClassProcessor.cs
index 89f90ae1..6745b46c 100644
--- a/Harmony/Public/PatchClassProcessor.cs
+++ b/Harmony/Public/PatchClassProcessor.cs
@@ -13,17 +13,17 @@ public class PatchClassProcessor
 
 		readonly Type containerType;
 		readonly HarmonyMethod containerAttributes;
-		readonly Dictionary<Type, MethodInfo> auxiliaryMethods;
+		readonly Dictionary<Type, MethodInfo> auxilaryMethods;
 
 		readonly List<AttributePatch> patchMethods;
 
-		static readonly List<Type> auxiliaryTypes = new()
-		{
+		static readonly List<Type> auxilaryTypes =
+		[
 			typeof(HarmonyPrepare),
 			typeof(HarmonyCleanup),
 			typeof(HarmonyTargetMethod),
 			typeof(HarmonyTargetMethods)
-		};
+		];
 
 		/// <summary name="Category">Name of the patch class's category</summary>
 		public string Category { get; set; }
@@ -51,11 +51,11 @@ public PatchClassProcessor(Harmony instance, Type type)
 			
 			Category = containerAttributes.category;
 
-			auxiliaryMethods = new Dictionary<Type, MethodInfo>();
-			foreach (var auxType in auxiliaryTypes)
+			auxilaryMethods = [];
+			foreach (var auxType in auxilaryTypes)
 			{
 				var method = PatchTools.GetPatchMethod(containerType, auxType.FullName);
-				if (method is not null) auxiliaryMethods[auxType] = method;
+				if (method is not null) auxilaryMethods[auxType] = method;
 			}
 
 			patchMethods = PatchTools.GetPatchMethods(containerType);
@@ -82,7 +82,7 @@ public List<MethodInfo> Patch()
 			{
 				RunMethod<HarmonyCleanup>(ref exception);
 				ReportException(exception, null);
-				return new List<MethodInfo>();
+				return [];
 			}
 
 			var replacements = new List<MethodInfo>();
@@ -235,7 +235,7 @@ List<MethodBase> GetBulkMethods()
 				else if (result.Any(m => m is null)) error = "some element was null";
 				if (error != null)
 				{
-					if (auxiliaryMethods.TryGetValue(typeof(HarmonyTargetMethods), out var method))
+					if (auxilaryMethods.TryGetValue(typeof(HarmonyTargetMethods), out var method))
 						throw new Exception($"Method {method.FullDescription()} returned an unexpected result: {error}");
 					else
 						throw new Exception($"Some method returned an unexpected result: {error}");
@@ -276,7 +276,7 @@ void ReportException(Exception exception, MethodBase original)
 
 		T RunMethod<S, T>(T defaultIfNotExisting, T defaultIfFailing, Func<T, string> failOnResult = null, params object[] parameters)
 		{
-			if (auxiliaryMethods.TryGetValue(typeof(S), out var method))
+			if (auxilaryMethods.TryGetValue(typeof(S), out var method))
 			{
 				var input = (parameters ?? new object[0]).Union(new object[] { instance }).ToArray();
 				var actualParameters = AccessTools.ActualParameters(method, input);
@@ -314,7 +314,7 @@ T RunMethod<S, T>(T defaultIfNotExisting, T defaultIfFailing, Func<T, string> fa
 
 		void RunMethod<S>(ref Exception exception, params object[] parameters)
 		{
-			if (auxiliaryMethods.TryGetValue(typeof(S), out var method))
+			if (auxilaryMethods.TryGetValue(typeof(S), out var method))
 			{
 				var input = (parameters ?? new object[0]).Union(new object[] { instance }).ToArray();
 				var actualParameters = AccessTools.ActualParameters(method, input);
diff --git a/Harmony/Tools/AccessTools.cs b/Harmony/Tools/AccessTools.cs
index 3721d04c..c7a9f719 100644
--- a/Harmony/Tools/AccessTools.cs
+++ b/Harmony/Tools/AccessTools.cs
@@ -7,8 +7,9 @@
 using System.Linq;
 using System.Reflection;
 using System.Reflection.Emit;
-using System.Runtime.Serialization;
 using System.Threading;
+using System.Runtime.Serialization;
+
 
 #if NET45_OR_GREATER || NETSTANDARD || NETCOREAPP
 using System.Runtime.CompilerServices;
@@ -659,7 +660,7 @@ public static List<string> GetMethodNames(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetMethodNames: type is null");
-				return new List<string>();
+				return [];
 			}
 			return GetDeclaredMethods(type).Select(m => m.Name).ToList();
 		}
@@ -673,7 +674,7 @@ public static List<string> GetMethodNames(object instance)
 			if (instance is null)
 			{
 				FileLog.Debug("AccessTools.GetMethodNames: instance is null");
-				return new List<string>();
+				return [];
 			}
 			return GetMethodNames(instance.GetType());
 		}
@@ -687,7 +688,7 @@ public static List<string> GetFieldNames(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetFieldNames: type is null");
-				return new List<string>();
+				return [];
 			}
 			return GetDeclaredFields(type).Select(f => f.Name).ToList();
 		}
@@ -701,7 +702,7 @@ public static List<string> GetFieldNames(object instance)
 			if (instance is null)
 			{
 				FileLog.Debug("AccessTools.GetFieldNames: instance is null");
-				return new List<string>();
+				return [];
 			}
 			return GetFieldNames(instance.GetType());
 		}
@@ -715,7 +716,7 @@ public static List<string> GetPropertyNames(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetPropertyNames: type is null");
-				return new List<string>();
+				return [];
 			}
 			return GetDeclaredProperties(type).Select(f => f.Name).ToList();
 		}
@@ -729,7 +730,7 @@ public static List<string> GetPropertyNames(object instance)
 			if (instance is null)
 			{
 				FileLog.Debug("AccessTools.GetPropertyNames: instance is null");
-				return new List<string>();
+				return [];
 			}
 			return GetPropertyNames(instance.GetType());
 		}
@@ -823,7 +824,7 @@ public static List<ConstructorInfo> GetDeclaredConstructors(Type type, bool? sea
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetDeclaredConstructors: type is null");
-				return new List<ConstructorInfo>();
+				return [];
 			}
 			var flags = allDeclared;
 			if (searchForStatic.HasValue)
@@ -840,7 +841,7 @@ public static List<MethodInfo> GetDeclaredMethods(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetDeclaredMethods: type is null");
-				return new List<MethodInfo>();
+				return [];
 			}
 			return type.GetMethods(allDeclared).ToList();
 		}
@@ -854,7 +855,7 @@ public static List<PropertyInfo> GetDeclaredProperties(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetDeclaredProperties: type is null");
-				return new List<PropertyInfo>();
+				return [];
 			}
 			return type.GetProperties(allDeclared).ToList();
 		}
@@ -868,7 +869,7 @@ public static List<FieldInfo> GetDeclaredFields(Type type)
 			if (type is null)
 			{
 				FileLog.Debug("AccessTools.GetDeclaredFields: type is null");
-				return new List<FieldInfo>();
+				return [];
 			}
 			return type.GetFields(allDeclared).ToList();
 		}
@@ -1865,7 +1866,11 @@ public static object CreateInstance(Type type)
 				CallingConventions.Any, new Type[0], modifiers: null);
 			if (ctor is not null)
 				return ctor.Invoke(null);
-			return FormatterServices.GetUninitializedObject(type);
+#if NET5_0_OR_GREATER
+			return System.Runtime.CompilerServices.RuntimeHelpers.GetUninitializedObject(type);
+#else
+			return System.Runtime.Serialization.FormatterServices.GetUninitializedObject(type);
+#endif
 		}
 
 		/// <summary>Creates an (possibly uninitialized) instance of a given type</summary>
@@ -1885,7 +1890,7 @@ public static T CreateInstance<T>()
 		/// A cache for the <see cref="ICollection{T}.Add"/> or similar Add methods for different types.
 		/// </summary>
 		///
-		static readonly Dictionary<Type, FastInvokeHandler> addHandlerCache = new();
+		static readonly Dictionary<Type, FastInvokeHandler> addHandlerCache = [];
 
 #if NET35
 		static readonly ReaderWriterLock addHandlerCacheLock = new();
diff --git a/Harmony/Tools/AccessToolsExtensions.cs b/Harmony/Tools/AccessToolsExtensions.cs
index 6043daa0..9b09fd7e 100644
--- a/Harmony/Tools/AccessToolsExtensions.cs
+++ b/Harmony/Tools/AccessToolsExtensions.cs
@@ -264,13 +264,13 @@ public static class AccessToolsExtensions
 		/// </param>
 		/// <param name="fieldName">The name of the field</param>
 		/// <returns>
-		/// A readable/assignable <see cref="FieldRef{T,F}"/> delegate with <c>T=object</c>
+		/// A readable/assignable <see cref="AccessTools.FieldRef{T,F}"/> delegate with <c>T=object</c>
 		/// (for static fields, the <c>instance</c> delegate parameter is ignored)
 		/// </returns>
 		/// <remarks>
 		/// <para>
 		/// This method is meant for cases where the given type is only known at runtime and thus can't be used as a type parameter <c>T</c>
-		/// in e.g. <see cref="FieldRefAccess{T, F}(string)"/>.
+		/// in e.g. <see cref="AccessTools.FieldRefAccess{T, F}(string)"/>.
 		/// </para>
 		/// <para>
 		/// This method supports static fields, even those defined in structs, for legacy reasons.
diff --git a/Harmony/Tools/CodeMatch.cs b/Harmony/Tools/CodeMatch.cs
index 6eef6f5c..0aab6db7 100644
--- a/Harmony/Tools/CodeMatch.cs
+++ b/Harmony/Tools/CodeMatch.cs
@@ -13,7 +13,7 @@ public class CodeMatch : CodeInstruction
 		public string name;
 
 		/// <summary>The matched opcodes</summary>
-		public HashSet<OpCode> opcodeSet = new();
+		public HashSet<OpCode> opcodeSet = [];
 
 		// for backwards compatibility we keep
 		/// <summary>The matched opcodes</summary>
@@ -27,13 +27,13 @@ public List<OpCode> opcodes
 #pragma warning restore IDE1006
 
 		/// <summary>The matched operands</summary>
-		public List<object> operands = new();
+		public List<object> operands = [];
 
 		/// <summary>The jumps from the match</summary>
-		public List<int> jumpsFrom = new();
+		public List<int> jumpsFrom = [];
 
 		/// <summary>The jumps to the match</summary>
-		public List<int> jumpsTo = new();
+		public List<int> jumpsTo = [];
 
 		/// <summary>The match predicate</summary>
 		public Func<CodeInstruction, bool> predicate;
diff --git a/Harmony/Tools/CodeMatcher.cs b/Harmony/Tools/CodeMatcher.cs
index 515109f5..5bbbae5f 100644
--- a/Harmony/Tools/CodeMatcher.cs
+++ b/Harmony/Tools/CodeMatcher.cs
@@ -10,14 +10,14 @@ namespace HarmonyLib
 	public class CodeMatcher
 	{
 		private readonly ILGenerator generator;
-		private readonly List<CodeInstruction> codes = new();
+		private readonly List<CodeInstruction> codes = [];
 
 		/// <summary>The current position</summary>
 		/// <value>The index or -1 if out of bounds</value>
 		///
 		public int Pos { get; private set; } = -1;
 
-		private Dictionary<string, CodeInstruction> lastMatches = new();
+		private Dictionary<string, CodeInstruction> lastMatches = [];
 		private string lastError;
 		private delegate CodeMatcher MatchDelegate();
 		private MatchDelegate lastMatchCall;
@@ -663,7 +663,7 @@ public CodeInstruction NamedMatch(string name)
 		private bool MatchSequence(int start, CodeMatch[] matches)
 		{
 			if (start < 0) return false;
-			lastMatches = new Dictionary<string, CodeInstruction>();
+			lastMatches = [];
 			foreach (var match in matches)
 			{
 				if (start >= Length || match.Matches(codes, codes[start]) == false)
diff --git a/Harmony/Tools/Extensions.cs b/Harmony/Tools/Extensions.cs
index 885ac75d..c8ff9ab2 100644
--- a/Harmony/Tools/Extensions.cs
+++ b/Harmony/Tools/Extensions.cs
@@ -165,62 +165,62 @@ public static string ToLiteral(this string input, string quoteChar = "\"")
 	///
 	public static class CodeInstructionExtensions
 	{
-		internal static readonly HashSet<OpCode> opcodesCalling = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesCalling =
+		[
 			OpCodes.Call,
 			OpCodes.Callvirt
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesLoadingLocalByAddress = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesLoadingLocalByAddress =
+		[
 			OpCodes.Ldloca_S,
 			OpCodes.Ldloca
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesLoadingLocalNormal = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesLoadingLocalNormal =
+		[
 			OpCodes.Ldloc_0,
 			OpCodes.Ldloc_1,
 			OpCodes.Ldloc_2,
 			OpCodes.Ldloc_3,
 			OpCodes.Ldloc_S,
 			OpCodes.Ldloc
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesStoringLocal = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesStoringLocal =
+		[
 			OpCodes.Stloc_0,
 			OpCodes.Stloc_1,
 			OpCodes.Stloc_2,
 			OpCodes.Stloc_3,
 			OpCodes.Stloc_S,
 			OpCodes.Stloc
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesLoadingArgumentByAddress = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesLoadingArgumentByAddress =
+		[
 			OpCodes.Ldarga_S,
 			OpCodes.Ldarga
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesLoadingArgumentNormal = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesLoadingArgumentNormal =
+		[
 			OpCodes.Ldarg_0,
 			OpCodes.Ldarg_1,
 			OpCodes.Ldarg_2,
 			OpCodes.Ldarg_3,
 			OpCodes.Ldarg_S,
 			OpCodes.Ldarg
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesStoringArgument = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesStoringArgument =
+		[
 			OpCodes.Starg_S,
 			OpCodes.Starg
-		};
+		];
 
-		internal static readonly HashSet<OpCode> opcodesBranching = new()
-		{
+		internal static readonly HashSet<OpCode> opcodesBranching =
+		[
 			OpCodes.Br_S,
 			OpCodes.Brfalse_S,
 			OpCodes.Brtrue_S,
@@ -247,10 +247,10 @@ public static class CodeInstructionExtensions
 			OpCodes.Bgt_Un,
 			OpCodes.Ble_Un,
 			OpCodes.Blt_Un
-		};
+		];
 
-		static readonly HashSet<OpCode> constantLoadingCodes = new()
-		{
+		static readonly HashSet<OpCode> constantLoadingCodes =
+		[
 			OpCodes.Ldc_I4_M1,
 			OpCodes.Ldc_I4_0,
 			OpCodes.Ldc_I4_1,
@@ -266,7 +266,7 @@ public static class CodeInstructionExtensions
 			OpCodes.Ldc_I8,
 			OpCodes.Ldc_R4,
 			OpCodes.Ldc_R8
-		};
+		];
 
 		/// <summary>Returns if an <see cref="OpCode"/> is initialized and valid</summary>
 		/// <param name="code">The <see cref="OpCode"/></param>
diff --git a/Harmony/Tools/FileLog.cs b/Harmony/Tools/FileLog.cs
index 03ed441e..4f8ec8ba 100644
--- a/Harmony/Tools/FileLog.cs
+++ b/Harmony/Tools/FileLog.cs
@@ -60,7 +60,7 @@ public static string LogPath
 		///
 		public static int indentLevel = 0;
 
-		static List<string> buffer = new();
+		static List<string> buffer = [];
 
 		static string IndentString()
 		{
@@ -112,7 +112,7 @@ public static List<string> GetBuffer(bool clear)
 			{
 				var result = buffer;
 				if (clear)
-					buffer = new List<string>();
+					buffer = [];
 				return result;
 			}
 		}
diff --git a/HarmonyTests/HarmonyTests.csproj b/HarmonyTests/HarmonyTests.csproj
index 1a0f2ff7..cc9fca38 100644
--- a/HarmonyTests/HarmonyTests.csproj
+++ b/HarmonyTests/HarmonyTests.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
-		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0</TargetFrameworks>
+		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0;net8.0</TargetFrameworks>
 		<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
 		<LangVersion>preview</LangVersion>
 		<IsPackable>false</IsPackable>
@@ -10,7 +10,7 @@
 
 	<!-- In .NET 5.0 Binary Formatters are off by default. Support is added for .NET 5.0+ for fallback JSON Serialization -->
 	<Choose>
-		<When Condition="'$(TargetFramework)'=='net5.0' Or '$(TargetFramework)'=='net6.0' Or '$(TargetFramework)'=='net7.0'">
+		<When Condition="'$(TargetFramework)'=='net5.0' Or '$(TargetFramework)'=='net6.0' Or '$(TargetFramework)'=='net7.0' Or '$(TargetFramework)'=='net8.0'">
 			<PropertyGroup>
 				<IsNET5OrGreater>true</IsNET5OrGreater>
 			</PropertyGroup>
diff --git a/HarmonyTests/IL/DynamicArgumentPatches.cs b/HarmonyTests/IL/DynamicArgumentPatches.cs
index 994e2f9e..ca8b0ae4 100644
--- a/HarmonyTests/IL/DynamicArgumentPatches.cs
+++ b/HarmonyTests/IL/DynamicArgumentPatches.cs
@@ -67,7 +67,7 @@ public static List<int> Test3(Vec3 v, List<int> list)
 		{
 			try
 			{
-				return new List<int>();
+				return [];
 			}
 			finally
 			{
@@ -78,7 +78,7 @@ public static List<int> Test3(Vec3 v, List<int> list)
 	[TestFixture, NonParallelizable]
 	public class DynamicArgumentPatches : TestLogger
 	{
-		static readonly List<string> log = new();
+		static readonly List<string> log = [];
 
 		static bool General(string typeName, int token, object instance, object[] args)
 		{
diff --git a/HarmonyTests/Patching/Assets/CombinedPatchClass.cs b/HarmonyTests/Patching/Assets/CombinedPatchClass.cs
index 42c12ee5..1046fa63 100644
--- a/HarmonyTests/Patching/Assets/CombinedPatchClass.cs
+++ b/HarmonyTests/Patching/Assets/CombinedPatchClass.cs
@@ -9,7 +9,13 @@ public class CombinedPatchClass
 		[MethodImpl(MethodImplOptions.NoInlining)]
 		public void Method1()
 		{
-			throw new Exception();
+			try
+			{
+			}
+			catch (Exception)
+			{
+				throw;
+			}
 		}
 
 		[MethodImpl(MethodImplOptions.NoInlining)]
@@ -21,7 +27,13 @@ public string Method2(string str)
 		[MethodImpl(MethodImplOptions.NoInlining)]
 		public void Method3(int n)
 		{
-			throw new Exception("" + n);
+			try
+			{
+			}
+			catch (Exception)
+			{
+				throw;
+			}
 		}
 	}
 
diff --git a/HarmonyTests/Patching/Assets/PatchClasses.cs b/HarmonyTests/Patching/Assets/PatchClasses.cs
index a1b96be3..e7312808 100644
--- a/HarmonyTests/Patching/Assets/PatchClasses.cs
+++ b/HarmonyTests/Patching/Assets/PatchClasses.cs
@@ -238,7 +238,7 @@ public class Class6
 		public List<object> Method6()
 		{
 			_ = DateTime.Now;
-			return new List<object>() { someFloat, someString, someStruct };
+			return [someFloat, someString, someStruct];
 		}
 	}
 
@@ -582,10 +582,10 @@ public static MethodInfo Prefix(MethodBase method)
 			return PatchTools.CreateMethod(
 				$"{method.Name}_Class11Patch_Prefix",
 				typeof(bool),
-				new() {
+				[
 					new KeyValuePair<string, Type>("__result", typeof(string).MakeByRefType()),
 					new KeyValuePair<string, Type>("dummy", typeof(int))
-				},
+				],
 				il =>
 				{
 					//load "patched" into __result
@@ -635,7 +635,7 @@ public List<string> FizzBuzz()
 
 	public class Class13<T> : IEnumerable<T>
 	{
-		readonly List<T> store = new();
+		readonly List<T> store = [];
 
 		[MethodImpl(MethodImplOptions.NoInlining)]
 		public void Add(T item)
@@ -669,7 +669,7 @@ public static void Prefix(object __originalMethod, ref int item)
 
 	public class Class14
 	{
-		public static List<string> state = new();
+		public static List<string> state = [];
 
 		[MethodImpl(MethodImplOptions.NoInlining)]
 		public bool Test(string s, KeyValuePair<string, int> p)
@@ -885,7 +885,7 @@ public IEnumerable<int> Method()
 	[HarmonyPatch(nameof(Finalizer_Patch_Order_Class.Method))]
 	public class Finalizer_Patch_Order_Patch
 	{
-		public static List<string> events = new();
+		public static List<string> events = [];
 
 		public static List<string> GetEvents()
 		{
@@ -894,7 +894,7 @@ public static List<string> GetEvents()
 
 		public static void ResetTest()
 		{
-			events = new List<string>();
+			events = [];
 		}
 
 		[HarmonyPrefix]
@@ -988,7 +988,7 @@ public string Method(int n)
 	[HarmonyPatch(nameof(Affecting_Original_Prefixes_Class.Method))]
 	public class Affecting_Original_Prefixes_Patch
 	{
-		public static List<string> events = new();
+		public static List<string> events = [];
 
 		public static List<string> GetEvents()
 		{
@@ -997,7 +997,7 @@ public static List<string> GetEvents()
 
 		public static void ResetTest()
 		{
-			events = new List<string>();
+			events = [];
 		}
 
 		[HarmonyPrefix]
diff --git a/HarmonyTests/Patching/Assets/Specials.cs b/HarmonyTests/Patching/Assets/Specials.cs
index 38f613f3..d4c56b86 100644
--- a/HarmonyTests/Patching/Assets/Specials.cs
+++ b/HarmonyTests/Patching/Assets/Specials.cs
@@ -47,9 +47,11 @@ static void Prefix()
 		{
 		}
 
-		static void Postfix()
-		{
-		}
+		// cannot patch method that ends in throw with a postfix
+		//
+		// static void Postfix()
+		// {
+		// }
 	}
 
 	[HarmonyPatch(typeof(DeadEndCode), nameof(DeadEndCode.Method))]
diff --git a/HarmonyTests/Patching/FinalizerPatches.cs b/HarmonyTests/Patching/FinalizerPatches.cs
index 552101bd..258beb77 100644
--- a/HarmonyTests/Patching/FinalizerPatches.cs
+++ b/HarmonyTests/Patching/FinalizerPatches.cs
@@ -229,7 +229,7 @@ public static void Patch()
 			var obj = Activator.CreateInstance(originalType);
 			var m_method = AccessTools.Method(originalType, "Method");
 			Assert.NotNull(m_method, nameof(m_method));
-			info = new Dictionary<string, object>();
+			info = [];
 			try
 			{
 				if (m_method.ReturnType == typeof(void))
diff --git a/HarmonyTests/Patching/GenericsPatches.cs b/HarmonyTests/Patching/GenericsPatches.cs
index d4c0d892..532b465b 100644
--- a/HarmonyTests/Patching/GenericsPatches.cs
+++ b/HarmonyTests/Patching/GenericsPatches.cs
@@ -67,7 +67,7 @@ void IEnumerator.Reset()
 			}
 		}
 
-		public List<T> list = new();
+		public List<T> list = [];
 
 		[MethodImpl(MethodImplOptions.NoInlining)]
 		public MyEnumerator GetEnumerator()
@@ -92,7 +92,7 @@ public class TestGenericStructReturnTypes_Patch
 	{
 		public static MyList<int>.MyEnumerator Postfix(MyList<int>.MyEnumerator input)
 		{
-			input.SetList(new List<int>() { 100, 200, 300 });
+			input.SetList([100, 200, 300]);
 			return input;
 		}
 	}
@@ -120,7 +120,7 @@ public void Test_GenericStructReturnTypes()
 			_ = patcher.AddPostfix(postfix);
 			_ = patcher.Patch();
 
-			var list = new MyList<int> { list = new List<int>() { 1, 2, 3 } };
+			var list = new MyList<int> { list = [1, 2, 3] };
 
 			var enumerator = list.GetEnumerator();
 			var result = new List<int>();
diff --git a/HarmonyTests/Patching/Transpiling.cs b/HarmonyTests/Patching/Transpiling.cs
index 537a8864..42b3eb2d 100644
--- a/HarmonyTests/Patching/Transpiling.cs
+++ b/HarmonyTests/Patching/Transpiling.cs
@@ -54,7 +54,7 @@ public static IEnumerable<CodeInstruction> TestTranspiler(IEnumerable<CodeInstru
 				if (instruction.opcode == insertLoc)
 				{
 					var blocks = instruction.blocks;
-					instruction.blocks = new List<ExceptionBlock>();
+					instruction.blocks = [];
 
 					var log = AccessTools.DeclaredField(typeof(Class3), "log");
 					var tst = typeof(string);
diff --git a/HarmonyTests/Tools/Assets/AccessToolsClass.cs b/HarmonyTests/Tools/Assets/AccessToolsClass.cs
index bf29450f..d535a600 100644
--- a/HarmonyTests/Tools/Assets/AccessToolsClass.cs
+++ b/HarmonyTests/Tools/Assets/AccessToolsClass.cs
@@ -58,7 +58,7 @@ public static IInner NewInnerStruct(int x)
 		private Inner field5 = new() { x = 999 };
 		private Inner[] field6 = new Inner[] { new Inner { x = 11 }, new Inner { x = 22 } };
 		private InnerStruct field7 = new() { x = 999 };
-		private List<InnerStruct> field8 = new() { new InnerStruct { x = 11 }, new InnerStruct { x = 22 } };
+		private List<InnerStruct> field8 = [new InnerStruct { x = 11 }, new InnerStruct { x = 22 }];
 		internal DayOfWeek field9 = DayOfWeek.Saturday;
 
 		private int _property = 314159;
diff --git a/HarmonyTests/Tools/TestFieldRefAccess.cs b/HarmonyTests/Tools/TestFieldRefAccess.cs
index 698e90c2..f92e93f9 100644
--- a/HarmonyTests/Tools/TestFieldRefAccess.cs
+++ b/HarmonyTests/Tools/TestFieldRefAccess.cs
@@ -939,7 +939,7 @@ public void Test_StructInstance_PublicString()
 				TestSuite_Struct<AccessToolsStruct, IComparable>(
 					field, "structField1test", expectedCaseToConstraint);
 				TestSuite_Struct<AccessToolsStruct, List<string>>(
-					field, new List<string> { "should always throw" }, IncompatibleFieldType(expectedCaseToConstraint));
+					field, ["should always throw"], IncompatibleFieldType(expectedCaseToConstraint));
 			});
 		}
 
diff --git a/HarmonyTests/Traverse/TestTraverse_Basics.cs b/HarmonyTests/Traverse/TestTraverse_Basics.cs
index ab5df586..150bbeea 100644
--- a/HarmonyTests/Traverse/TestTraverse_Basics.cs
+++ b/HarmonyTests/Traverse/TestTraverse_Basics.cs
@@ -9,7 +9,7 @@ namespace HarmonyLibTests.Tools
 	[TestFixture, NonParallelizable]
 	public class TestTraverse_Basics : TestLogger
 	{
-		static readonly List<string> fieldNames = new() { "_root", "_type", "_info", "_method", "_params" };
+		static readonly List<string> fieldNames = ["_root", "_type", "_info", "_method", "_params"];
 
 		// Basic integrity check for our test class and the field-testvalue relations
 		//
diff --git a/TestLibrary/TestLibrary.csproj b/TestLibrary/TestLibrary.csproj
index 89807d45..64f99e75 100644
--- a/TestLibrary/TestLibrary.csproj
+++ b/TestLibrary/TestLibrary.csproj
@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
-		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0</TargetFrameworks>
+		<TargetFrameworks>net35;net452;net472;net48;netcoreapp3.0;netcoreapp3.1;net5.0;net6.0;net7.0;net8.0</TargetFrameworks>
 		<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
 		<LangVersion>preview</LangVersion>
 		<IsPackable>false</IsPackable>
diff --git a/TestLibrary/TestPatch.cs b/TestLibrary/TestPatch.cs
index 48074b72..dace79de 100644
--- a/TestLibrary/TestPatch.cs
+++ b/TestLibrary/TestPatch.cs
@@ -24,7 +24,7 @@ private static void Postfix(ref Assembly postfix)
 		postfix = Assembly.GetExecutingAssembly();
 	}
 
-	private static void Finalizer(Exception __exception) { }
+	private static void Finalizer(Exception __exception) { _ = __exception; }
 
 	private static IEnumerable<CodeInstruction> EmptyTranspiler(IEnumerable<CodeInstruction> instructions) => instructions;
 
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
index ec90b027..50793693 100644
--- a/azure-pipelines.yml
+++ b/azure-pipelines.yml
@@ -13,7 +13,7 @@ jobs:
   parameters:
     os: windows
     architecture: x64
-    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0]
+    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0, net8.0]
     publishBuild: true
 
 - template: azure-pipelines-job-template.yml
@@ -21,7 +21,7 @@ jobs:
     # DependsOn: windows_dotnet_x64
     os: windows
     architecture: x86
-    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0]
+    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0, net8.0]
 
 - template: azure-pipelines-job-template.yml
   parameters:
@@ -44,7 +44,7 @@ jobs:
     # DependsOn: windows_mono_x86
     os: ubuntu
     architecture: x64
-    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0]
+    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0, net8.0]
 
 # Without an x86 ubuntu agent and with mono's packaging not multi-arch-safe (can't install x86 mono on x64 ubuntu), we can't test on x86 ubuntu.
 #- template: azure-pipelines-job-template.yml
@@ -58,7 +58,7 @@ jobs:
     # DependsOn: ubuntu_dotnet_x64
     os: macOS
     architecture: x64
-    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0]
+    frameworks: [net35, net452, net472, net48, netcoreapp3.0, netcoreapp3.1, net5.0, net6.0, net7.0, net8.0]
 
 # x86 ubuntu does apply to x86 macOS as well.
 #- template: azure-pipelines-job-template.yml
diff --git a/testEnvironments.json b/testEnvironments.json
new file mode 100644
index 00000000..5264ee48
--- /dev/null
+++ b/testEnvironments.json
@@ -0,0 +1,10 @@
+{
+  "version": "1",
+  "environments": [
+    {
+      "name": "WSL Ubuntu",
+      "type": "wsl",
+      "wslDistribution": "Ubuntu"
+    }
+  ]
+}
\ No newline at end of file