Fixed potential performance issue with for-loops when Spans are passed by reference

RGB.NET.Core/Rendering/Textures/PixelTexture.cs

@@ -126,7 +126,7 @@ public abstract class PixelTexture<T> : ITexture
     /// </summary>
     /// <param name="pixel">The pixel-data to convert.</param>
     /// <returns>The color represented by the specified pixel-data.</returns>
-    protected abstract Color GetColor(in ReadOnlySpan<T> pixel);
+    protected abstract Color GetColor(ReadOnlySpan<T> pixel);
 
     /// <summary>
     /// Gets the pixel-data at the specified location.
@@ -189,7 +189,7 @@ public sealed class PixelTexture : PixelTexture<Color>
 
     /// <inheritdoc />
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    protected override Color GetColor(in ReadOnlySpan<Color> pixel) => pixel[0];
+    protected override Color GetColor(ReadOnlySpan<Color> pixel) => pixel[0];
 
     #endregion
 }

RGB.NET.Core/Rendering/Textures/Sampler/AverageColorSampler.cs

@@ -22,7 +22,7 @@ public sealed class AverageColorSampler : ISampler<Color>
     #region Methods
 
     /// <inheritdoc />
-    public unsafe void Sample(in SamplerInfo<Color> info, in Span<Color> pixelData)
+    public unsafe void Sample(in SamplerInfo<Color> info, Span<Color> pixelData)
     {
         int count = info.Width * info.Height;
         if (count == 0) return;

RGB.NET.Core/Rendering/Textures/Sampler/ISampler.cs

@@ -13,5 +13,5 @@ public interface ISampler<T>
     /// </summary>
     /// <param name="info">The information containing the data to sample.</param>
     /// <param name="pixelData">The buffer used to write the resulting pixel to.</param>
-    void Sample(in SamplerInfo<T> info, in Span<T> pixelData);
+    void Sample(in SamplerInfo<T> info, Span<T> pixelData);
 }

RGB.NET.Core/Rendering/Textures/Sampler/SamplerInfo.cs

@@ -44,7 +44,7 @@ public readonly ref struct SamplerInfo<T>
     /// <param name="width">The width of the region the data comes from.</param>
     /// <param name="height">The height of region the data comes from.</param>
     /// <param name="data">The data to sample.</param>
-    public SamplerInfo(int x, int y, int width, int height, int stride, int dataPerPixel, in ReadOnlySpan<T> data)
+    public SamplerInfo(int x, int y, int width, int height, int stride, int dataPerPixel, ReadOnlySpan<T> data)
     {
         this._x = x;
         this._y = y;

RGB.NET.Core/Update/Devices/UpdateQueue.cs

@@ -80,7 +80,7 @@ public abstract class UpdateQueue<TIdentifier, TData> : AbstractReferenceCountin
     /// Performs the update this queue is responsible for.
     /// </summary>
     /// <param name="dataSet">The set of data that needs to be updated.</param>
-    protected abstract bool Update(in ReadOnlySpan<(TIdentifier key, TData color)> dataSet);
+    protected abstract bool Update(ReadOnlySpan<(TIdentifier key, TData color)> dataSet);
 
     /// <summary>
     /// Sets or merges the provided data set in the current dataset and notifies the trigger that there is new data available.

RGB.NET.Devices.Asus/Generic/AsusUpdateQueue.cs

@@ -43,7 +43,7 @@ public sealed class AsusUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.CoolerMaster/Generic/CoolerMasterUpdateQueue.cs

@@ -37,7 +37,7 @@ public sealed class CoolerMasterUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Corsair/Generic/CorsairDeviceUpdateQueue.cs

@@ -40,7 +40,7 @@ public sealed class CorsairDeviceUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override unsafe bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override unsafe bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Corsair_Legacy/Generic/CorsairDeviceUpdateQueue.cs

@@ -35,7 +35,7 @@ public class CorsairDeviceUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.DMX/E131/E131UpdateQueue.cs

@@ -59,7 +59,7 @@ public sealed class E131UpdateQueue : UpdateQueue
     }
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Debug/DebugDeviceUpdateQueue.cs

@@ -7,7 +7,7 @@ internal sealed class DebugDeviceUpdateQueue() : UpdateQueue(new DeviceUpdateTri
 {
     #region Methods
 
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet) => true;
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet) => true;
 
     #endregion
 }

RGB.NET.Devices.Logitech/PerDevice/LogitechPerDeviceUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class LogitechPerDeviceUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Logitech/PerKey/LogitechPerKeyUpdateQueue.cs

@@ -24,7 +24,7 @@ public sealed class LogitechPerKeyUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Logitech/Zone/LogitechZoneUpdateQueue.cs

@@ -33,7 +33,7 @@ public sealed class LogitechZoneUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Msi/Generic/MsiDeviceUpdateQueue.cs

@@ -34,7 +34,7 @@ public sealed class MsiDeviceUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Novation/Generic/MidiUpdateQueue.cs

@@ -35,7 +35,7 @@ public abstract class MidiUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.OpenRGB/Generic/OpenRGBUpdateQueue.cs

@@ -48,7 +48,7 @@ public sealed class OpenRGBUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.PicoPi/PicoPi/PicoPiBulkUpdateQueue.cs

@@ -44,7 +44,7 @@ public sealed class PicoPiBulkUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.PicoPi/PicoPi/PicoPiHIDUpdateQueue.cs

@@ -41,7 +41,7 @@ public sealed class PicoPiHIDUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.PicoPi/PicoPi/PicoPiSDK.cs

@@ -212,7 +212,7 @@ public sealed class PicoPiSDK : IDisposable
     /// </summary>
     /// <param name="data">The data to send.</param>
     /// <param name="channel">The channel to update.</param>
-    public void SendHidUpdate(in Span<byte> buffer, int channel)
+    public void SendHidUpdate(Span<byte> buffer, int channel)
     {
         int chunks = buffer.Length / HID_OFFSET_MULTIPLIER;
         if ((chunks * HID_OFFSET_MULTIPLIER) < buffer.Length) chunks++;
@@ -232,7 +232,7 @@ public sealed class PicoPiSDK : IDisposable
     /// <param name="channel">The channel to update.</param>
     /// <param name="chunk">The chunk id of the packet. (Required if packets are fragmented.)</param>
     /// <param name="update">A value indicating if the device should update directly after receiving this packet. (If packets are fragmented this should only be true for the last chunk.)</param>
-    public void SendHidUpdate(in Span<byte> data, int channel, int chunk, bool update)
+    public void SendHidUpdate(Span<byte> data, int channel, int chunk, bool update)
     {
         if (data.Length == 0) return;
 
@@ -253,7 +253,7 @@ public sealed class PicoPiSDK : IDisposable
     /// </remarks>
     /// <param name="data">The data packet to send.</param>
     /// <param name="channel">The channel to update.</param>
-    public void SendBulkUpdate(in Span<byte> data, int channel)
+    public void SendBulkUpdate(Span<byte> data, int channel)
     {
         if ((data.Length == 0) || !IsBulkSupported) return;
 

RGB.NET.Devices.Razer/ChromaLink/RazerChromaLinkUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerChromaLinkUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.CHROMALINK_MAX_LEDS];
 

RGB.NET.Devices.Razer/Generic/RazerUpdateQueue.cs

@@ -30,7 +30,7 @@ public abstract class RazerUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {
@@ -77,7 +77,7 @@ public abstract class RazerUpdateQueue : UpdateQueue
     /// </summary>
     /// <param name="dataSet">The data to be updated.</param>
     /// <returns>An <see cref="IntPtr"/> pointing to the effect parameter struct.</returns>
-    protected abstract nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet);
+    protected abstract nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet);
 
     #endregion
 }

RGB.NET.Devices.Razer/Headset/RazerHeadsetUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerHeadsetUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.HEADSET_MAX_LEDS];
 

RGB.NET.Devices.Razer/Keyboard/RazerKeyboardUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerKeyboardUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.KEYBOARD_MAX_LEDS];
 

RGB.NET.Devices.Razer/Keypad/RazerKeypadUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerKeypadUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.KEYPAD_MAX_LEDS];
 

RGB.NET.Devices.Razer/Mouse/RazerMouseUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerMouseUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.MOUSE_MAX_LEDS];
 

RGB.NET.Devices.Razer/Mousepad/RazerMousepadUpdateQueue.cs

@@ -25,7 +25,7 @@ public sealed class RazerMousepadUpdateQueue : RazerUpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override nint CreateEffectParams(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override nint CreateEffectParams(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         _Color[] colors = new _Color[_Defines.MOUSEPAD_MAX_LEDS];
 

RGB.NET.Devices.SteelSeries/Generic/SteelSeriesDeviceUpdateQueue.cs

@@ -51,7 +51,7 @@ internal sealed class SteelSeriesDeviceUpdateQueue : UpdateQueue
     }
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.WLED/Generic/WLedDeviceUpdateQueue.cs

@@ -63,7 +63,7 @@ internal sealed class WledDeviceUpdateQueue : UpdateQueue
     }
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.WS281X/Generic/SerialPortUpdateQueue.cs

@@ -56,7 +56,7 @@ public abstract class SerialConnectionUpdateQueue<TData> : UpdateQueue
     }
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.WS281X/NodeMCU/NodeMCUWS2812USBUpdateQueue.cs

@@ -77,7 +77,7 @@ public sealed class NodeMCUWS2812USBUpdateQueue : UpdateQueue
     }
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Devices.Wooting/Generic/WootingUpdateQueue.cs

@@ -33,7 +33,7 @@ public sealed class WootingUpdateQueue : UpdateQueue
     #region Methods
 
     /// <inheritdoc />
-    protected override bool Update(in ReadOnlySpan<(object key, Color color)> dataSet)
+    protected override bool Update(ReadOnlySpan<(object key, Color color)> dataSet)
     {
         try
         {

RGB.NET.Presets/Textures/BytePixelTexture.cs

@@ -61,7 +61,7 @@ public sealed class BytePixelTexture : PixelTexture<byte>
 
     /// <inheritdoc />
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    protected override Color GetColor(in ReadOnlySpan<byte> pixel)
+    protected override Color GetColor(ReadOnlySpan<byte> pixel)
     {
         return ColorFormat switch
         {

RGB.NET.Presets/Textures/FloatPixelTexture.cs

@@ -61,7 +61,7 @@ public sealed class FloatPixelTexture : PixelTexture<float>
 
     /// <inheritdoc />
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    protected override Color GetColor(in ReadOnlySpan<float> pixel)
+    protected override Color GetColor(ReadOnlySpan<float> pixel)
     {
         return ColorFormat switch
         {

RGB.NET.Presets/Textures/Sampler/AverageByteSampler.cs

@@ -18,7 +18,7 @@ public sealed class AverageByteSampler : ISampler<byte>
     #region Methods
 
     /// <inheritdoc />
-    public unsafe void Sample(in SamplerInfo<byte> info, in Span<byte> pixelData)
+    public unsafe void Sample(in SamplerInfo<byte> info, Span<byte> pixelData)
     {
         int count = info.Width * info.Height;
         if (count == 0) return;

RGB.NET.Presets/Textures/Sampler/AverageFloatSampler.cs

@@ -12,7 +12,7 @@ public sealed class AverageFloatSampler : ISampler<float>
     #region Methods
 
     /// <inheritdoc />
-    public unsafe void Sample(in SamplerInfo<float> info, in Span<float> pixelData)
+    public unsafe void Sample(in SamplerInfo<float> info, Span<float> pixelData)
     {
         int count = info.Width * info.Height;
         if (count == 0) return;