Fix new OpenRGB size checks

Author: diogotr7

src/OpenRGB.NET/OpenRGBClient.cs

@@ -21,7 +21,7 @@ public sealed class OpenRgbClient : IDisposable, IOpenRgbClient
 
     /// <inheritdoc />
     public ProtocolVersion MaxSupportedProtocolVersion => ProtocolVersion.FromNumber(MaxProtocolNumber);
-    
+
     /// <inheritdoc />
     public ProtocolVersion ClientProtocolVersion => ProtocolVersion.FromNumber(_protocolVersionNumber);
 
@@ -130,7 +130,7 @@ public void UpdateLeds(int deviceId, ReadOnlySpan<Color> colors)
 
         var bytes = MemoryMarshal.Cast<Color, byte>(colors);
 
-        _connection.Send(CommandId.UpdateLeds, (uint)deviceId, new Args<uint, ushort>(0, (ushort)colors.Length), bytes);
+        _connection.Send(CommandId.UpdateLeds, (uint)deviceId, new UpdateLedsArg((ushort)colors.Length), bytes);
     }
 
     /// <inheritdoc />
@@ -147,7 +147,7 @@ public void UpdateZoneLeds(int deviceId, int zoneId, ReadOnlySpan<Color> colors)
 
         var bytes = MemoryMarshal.Cast<Color, byte>(colors);
 
-        _connection.Send(CommandId.UpdateZoneLeds, (uint)deviceId, new Args<uint, uint, ushort>(0, (uint)zoneId, (ushort)colors.Length), bytes);
+        _connection.Send(CommandId.UpdateZoneLeds, (uint)deviceId, new UpdateZoneLedsArg((uint)zoneId, (ushort)colors.Length), bytes);
     }
 
     /// <inheritdoc />
@@ -223,7 +223,7 @@ public void UpdateMode(int deviceId, int modeId,
             targetMode.SetColors(colors);
         }
 
-        _connection.Send(CommandId.UpdateMode, (uint)deviceId, new ModeOperationArg(0, (uint)modeId, new ModeArg(targetMode)));
+        _connection.Send(CommandId.UpdateMode, (uint)deviceId, new ModeOperationArg((uint)modeId, new ModeArg(targetMode)));
     }
 
     /// <inheritdoc />
@@ -236,7 +236,7 @@ public void SaveMode(int deviceId, int modeId)
 
         var targetMode = targetDevice.Modes[modeId];
 
-        _connection.Send(CommandId.SaveMode, (uint)deviceId, new ModeOperationArg(0, (uint)modeId, new ModeArg(targetMode)));
+        _connection.Send(CommandId.SaveMode, (uint)deviceId, new ModeOperationArg((uint)modeId, new ModeArg(targetMode)));
     }
 
     /// <inheritdoc />

src/OpenRGB.NET/Writables/Args.cs

@@ -22,18 +22,4 @@ public void WriteTo(ref SpanWriter writer)
         writer.Write(Arg1);
         writer.Write(Arg2);
     }
-}
-
-internal readonly record struct Args<T1, T2, T3>(T1 Arg1, T2 Arg2, T3 Arg3) : ISpanWritable 
-    where T1 : unmanaged
-    where T2 : unmanaged
-    where T3 : unmanaged
-{
-    public int Length => Unsafe.SizeOf<T1>() + Unsafe.SizeOf<T2>() + Unsafe.SizeOf<T3>();
-    public void WriteTo(ref SpanWriter writer)
-    {
-        writer.Write(Arg1);
-        writer.Write(Arg2);
-        writer.Write(Arg3);
-    }
 }

src/OpenRGB.NET/Writables/ModeOperationArg.cs

@@ -2,12 +2,12 @@
 
 namespace OpenRGB.NET;
 
-internal readonly record struct ModeOperationArg(uint DataSize, uint ModeId, ModeArg Mode) : ISpanWritable
+internal readonly record struct ModeOperationArg(uint ModeId, ModeArg Mode) : ISpanWritable
 {
     public int Length => sizeof(uint) * 2 + Mode.Length;
     public void WriteTo(ref SpanWriter writer)
     {
-        writer.Write(DataSize);
+        writer.Write(Length);
         writer.Write(ModeId);
         Mode.WriteTo(ref writer);
     }

src/OpenRGB.NET/Writables/UpdateLedsArg.cs

@@ -0,0 +1,17 @@
+using System.Runtime.CompilerServices;
+using OpenRGB.NET.Utils;
+
+namespace OpenRGB.NET;
+
+internal readonly record struct UpdateLedsArg(ushort ColorCount) : ISpanWritable
+{
+    public int Length => sizeof(uint) +
+                         sizeof(uint) +
+                         sizeof(ushort);
+    
+    public void WriteTo(ref SpanWriter writer)
+    {
+        writer.Write((uint)(Length + Unsafe.SizeOf<Color>() * ColorCount));
+        writer.Write(ColorCount);
+    }
+}

src/OpenRGB.NET/Writables/UpdateZoneLedsArg.cs

@@ -0,0 +1,18 @@
+using System.Runtime.CompilerServices;
+using OpenRGB.NET.Utils;
+
+namespace OpenRGB.NET;
+
+internal readonly record struct UpdateZoneLedsArg(uint ZoneId, ushort ColorCount) : ISpanWritable
+{
+    public int Length => sizeof(uint) +
+                         sizeof(uint) +
+                         sizeof(ushort);
+    
+    public void WriteTo(ref SpanWriter writer)
+    {
+        writer.Write((uint)(Length + Unsafe.SizeOf<Color>() * ColorCount));
+        writer.Write(ZoneId);
+        writer.Write(ColorCount);
+    }
+}