Added method to opt in to the keypress-callback since this prevents reinitializing

Author: DarthAffe

CueSDK.cs

@@ -104,7 +104,7 @@ public static partial class CueSDK
 
         [UnmanagedFunctionPointer(CallingConvention.Cdecl)]
         private delegate void OnKeyPressedDelegate(IntPtr context, CorsairKeyId keyId, [MarshalAs(UnmanagedType.I1)] bool pressed);
-        private static readonly OnKeyPressedDelegate _onKeyPressedDelegate = OnKeyPressed;
+        private static OnKeyPressedDelegate _onKeyPressedDelegate;
 
         #endregion
 
@@ -113,6 +113,8 @@ public static partial class CueSDK
         /// <summary>
         /// Occurs when the SDK reports that a key is pressed.
         /// Notice that right now only G- (keyboard) and M- (mouse) keys are supported.
+        /// 
+        /// To enable this event <see cref="EnableKeypressCallback"/> needs to be called.
         /// </summary>
         public static event EventHandler<KeyPressedEventArgs> KeyPressed;
 
@@ -247,7 +249,6 @@ public static void Initialize(bool exclusiveAccess = false)
                     Throw(error, true);
             }
 
-            _CUESDK.CorsairRegisterKeypressCallback(Marshal.GetFunctionPointerForDelegate(_onKeyPressedDelegate), IntPtr.Zero);
             error = LastError;
             if (error != CorsairError.Success)
                 Throw(error, false);
@@ -257,6 +258,21 @@ public static void Initialize(bool exclusiveAccess = false)
             IsInitialized = true;
         }
 
+        /// <summary>
+        /// Enables the keypress-callback.
+        /// This method needs to be called to enable the <see cref="KeyPressed"/>-event.
+        /// 
+        /// WARNING: AFTER THIS METHOD IS CALLED IT'S NO LONGER POSSIBLE TO REINITIALIZE THE SDK!
+        /// </summary>
+        public static void EnableKeypressCallback()
+        {
+            if (!IsInitialized)
+                throw new WrapperException("CueSDK isn't initialized.");
+
+            _onKeyPressedDelegate = OnKeyPressed;
+            _CUESDK.CorsairRegisterKeypressCallback(Marshal.GetFunctionPointerForDelegate(_onKeyPressedDelegate), IntPtr.Zero);
+        }
+
         /// <summary>
         /// Resets the colors of all devices back to the last saved color-data. (If there wasn't a manual save, that's the data from the time the SDK was initialized.)
         /// </summary>
@@ -283,6 +299,9 @@ public static void Reinitialize(bool exclusiveAccess)
             if (!IsInitialized)
                 throw new WrapperException("CueSDK isn't initialized.");
 
+            if (_onKeyPressedDelegate != null)
+                throw new WrapperException("Keypress-Callback is enabled.");
+
             KeyboardSDK?.ResetLeds();
             MouseSDK?.ResetLeds();
             HeadsetSDK?.ResetLeds();
@@ -343,7 +362,6 @@ public static void Reinitialize(bool exclusiveAccess)
                     || HeadsetStandSDK.HeadsetStandDeviceInfo.Model != reloadedDevices[CorsairDeviceType.HeadsetStand].Model)
                     throw new WrapperException("The previously loaded Headset Stand got disconnected.");
 
-            _CUESDK.CorsairRegisterKeypressCallback(Marshal.GetFunctionPointerForDelegate(_onKeyPressedDelegate), IntPtr.Zero);
             error = LastError;
             if (error != CorsairError.Success)
                 Throw(error, false);

Examples/SimpleDevTest/Program.cs

@@ -39,6 +39,8 @@ public static void Main(string[] args)
                 CueSDK.Initialize();
                 Console.WriteLine("Initialized with " + CueSDK.LoadedArchitecture + "-SDK");
 
+                CueSDK.EnableKeypressCallback();
+
                 CueSDK.KeyPressed += (sender, eventArgs) => Console.WriteLine($"Key {eventArgs.KeyId} {(eventArgs.IsPressed ? "pressed" : "released")}");
 
                 //CueSDK.KeyboardSDK.Brush = (SolidColorBrush)Color.Black;
@@ -76,6 +78,17 @@ public static void Main(string[] args)
 
                 CueSDK.UpdateMode = UpdateMode.Continuous;
 
+                Wait(5);
+                CueSDK.UpdateMode = UpdateMode.Manual;
+                for (int i = 0; i < 100000; i++)
+                {
+                    CueSDK.Reinitialize();
+                    Console.WriteLine(i);
+                }
+
+                Console.WriteLine("done!");
+                Wait(5);
+
                 //IBrush rainbowBrush = new LinearGradientBrush(new RainbowGradient());
                 //rainbowBrush.AddEffect(new FlashEffect { Attack = 5f, Sustain = 1f, Decay = 0, Release = 5f, Interval = 1f });
                 //rainbowBrush.AddEffect(new MoveRainbowEffect());