Fix trimming errors added in net9

Author: xPaw

ValveKeyValue/ValveKeyValue/ObjectCopier.cs

@@ -19,6 +19,7 @@ public static object MakeObject(
             [DynamicallyAccessedMembers(Trimming.Properties)] Type objectType, KVObject keyValueObject, IObjectReflector reflector)
             => InvokeGeneric(nameof(MakeObject), objectType, new object[] { keyValueObject, reflector });
 
+        [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2062", Justification = "If the lookup value type exists at runtime then it should have enough for us to introspect.")]
         public static TObject MakeObject<[DynamicallyAccessedMembers(Trimming.Constructors | Trimming.Properties)] TObject>(KVObject keyValueObject, IObjectReflector reflector)
         {
             Require.NotNull(keyValueObject, nameof(keyValueObject));
@@ -46,7 +47,7 @@ public static object MakeObject(
                 // The object must remain boxed until it is fully initiallized, as this is the only way
                 // that we can build a struct due to the nature of struct copying.
                 var typedObject = RuntimeHelpers.GetUninitializedObject(typeof(TObject));
-                CopyObject(keyValueObject, typedObject, reflector);
+                CopyObject(keyValueObject, typeof(TObject), typedObject, reflector);
                 return (TObject)typedObject;
             }
             else if (TryConvertValueTo<TObject>(keyValueObject.Name, keyValueObject.Value, out var converted))
@@ -86,6 +87,7 @@ static KVObject FromObjectCore(
             return new KVObject(topLevelName, transformedValue);
         }
 
+        [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2072", Justification = "If the IDictionary's value object already exists at runtime then its properties will too.")]
         static KVValue ConvertObjectToValue(
             [DynamicallyAccessedMembers(Trimming.Properties)] Type objectType,
             object managedObject,
@@ -144,7 +146,7 @@ static KVValue ConvertObjectToValue(
             return childObjects;
         }
 
-        static void CopyObject(KVObject kv, object obj, IObjectReflector reflector)
+        static void CopyObject(KVObject kv, [DynamicallyAccessedMembers(Trimming.Properties)] Type objectType, object obj, IObjectReflector reflector)
         {
             Require.NotNull(kv, nameof(kv));
 
@@ -155,7 +157,7 @@ static void CopyObject(KVObject kv, object obj, IObjectReflector reflector)
 
             Require.NotNull(reflector, nameof(reflector));
 
-            var members = reflector.GetMembers(obj.GetType(), obj).ToDictionary(m => m.Name, m => m, StringComparer.OrdinalIgnoreCase);
+            var members = reflector.GetMembers(objectType, obj).ToDictionary(m => m.Name, m => m, StringComparer.OrdinalIgnoreCase);
 
             foreach (var item in kv.Children)
             {
@@ -374,8 +376,7 @@ static bool IsDictionary(Type type)
         [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2060", Justification = "Analysis cannot follow MakeGenericMethod but we should be clear by here anyway.")]
         [UnconditionalSuppressMessage("AOT", "IL3050", Justification = "Analysis cannot follow MakeGenericMethod but we should be clear by here anyway.")]
         static object MakeDictionary(
-            [DynamicallyAccessedMembers(Trimming.Constructors | Trimming.Properties)]
-            Type type,
+            [DynamicallyAccessedMembers(Trimming.Constructors | Trimming.Properties)] Type type,
             KVObject kv,
             IObjectReflector reflector)
         {