Explicitly set SizeConst = 1 and remove the warning suppression (#90174)
authorJeremy Koritzinsky <jekoritz@microsoft.com>
Thu, 10 Aug 2023 18:31:40 +0000 (11:31 -0700)
committerGitHub <noreply@github.com>
Thu, 10 Aug 2023 18:31:40 +0000 (11:31 -0700)
src/libraries/System.Reflection.MetadataLoadContext/tests/src/SampleMetadata/SampleMetadata.cs

index 861e54b..a756898 100644 (file)
@@ -377,24 +377,14 @@ namespace SampleMetadata
         [MarshalAs(UnmanagedType.IDispatch, IidParameterIndex = 42)]
         public int F4;
 
-        // We are explicitly testing ByValArray without a SizeConst
-        // CS9125: Attribute parameter 'SizeConst' must be specified.
-
-#pragma warning disable CS9125
-        [MarshalAs(UnmanagedType.ByValArray)]
+        [MarshalAs(UnmanagedType.ByValArray, SizeConst = 1)]
         public int F5;
-#pragma warning restore CS9125
 
         [MarshalAs(UnmanagedType.ByValArray, SizeConst = 5)]
         public int F6;
 
-        // We are explicitly testing ByValArray without a SizeConst
-        // CS9125: Attribute parameter 'SizeConst' must be specified.
-
-#pragma warning disable CS9125
-        [MarshalAs(UnmanagedType.ByValArray, ArraySubType = UnmanagedType.FunctionPtr)]
+        [MarshalAs(UnmanagedType.ByValArray, SizeConst = 1, ArraySubType = UnmanagedType.FunctionPtr)]
         public int F7;
-#pragma warning restore CS9125
 
         [MarshalAs(UnmanagedType.ByValArray, SizeConst = 87, ArraySubType = UnmanagedType.FunctionPtr)]
         public int F8;