Remove unnecessary 'fixed' statement

GrabYourPitchforks · GrabYourPitchforks · commit 7b2aa639d2fe · 2025-03-14T16:38:09.000-07:00
diff --git a/src/System.Drawing.Common/src/System/Drawing/Imaging/EncoderParameter.cs b/src/System.Drawing.Common/src/System/Drawing/Imaging/EncoderParameter.cs
@@ -214,12 +214,9 @@ public EncoderParameter(Encoder encoder, long[] value)
         _parameterValue = Marshal.AllocHGlobal(checked(_numberOfValues * sizeof(int)));
 
         int* dest = (int*)_parameterValue;
-        fixed (long* source = value)
+        for (int i = 0; i < value.Length; i++)
         {
-            for (int i = 0; i < value.Length; i++)
-            {
-                dest[i] = (int)source[i];
-            }
+            dest[i] = (int)value[i];
         }
 
         GC.KeepAlive(this);

Original file line number	Diff line number	Diff line change
`@@ -214,12 +214,9 @@ public EncoderParameter(Encoder encoder, long[] value)`
`214`	`214`	`_parameterValue = Marshal.AllocHGlobal(checked(_numberOfValues * sizeof(int)));`
`215`	`215`
`216`	`216`	`int* dest = (int*)_parameterValue;`
`217`		`- fixed (long* source = value)`
	`217`	`+ for (int i = 0; i < value.Length; i++)`
`218`	`218`	`{`
`219`		`- for (int i = 0; i < value.Length; i++)`
`220`		`- {`
`221`		`- dest[i] = (int)source[i];`
`222`		`- }`
	`219`	`+ dest[i] = (int)value[i];`
`223`	`220`	`}`
`224`	`221`
`225`	`222`	`GC.KeepAlive(this);`