update LibreHardwareMonitorLib version because of false positive

optimize Razer Chroma Layer

Author: Aytackydln

Project-Aurora/Project-Aurora/Project-Aurora.csproj

@@ -75,7 +75,7 @@
     <PackageReference Include="Infragistics.Themes.MetroDark.Wpf" Version="1.0.0" />
     <PackageReference Include="IronPython" Version="2.7.11" />
     <PackageReference Include="JsonSubTypes" Version="1.8.0" />
-    <PackageReference Include="LibreHardwareMonitorLib" Version="0.8.8" />
+    <PackageReference Include="LibreHardwareMonitorLib" Version="0.8.9" />
     <PackageReference Include="MouseKeyHook" Version="5.6.0" />
     <PackageReference Include="NAudio" Version="2.0.1" />
     <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />

Project-Aurora/Project-Aurora/Settings/Layers/RazerLayerHandler.cs

@@ -63,12 +63,12 @@ public override void Default()
     [LayerHandlerMeta(Name = "Razer Chroma", IsDefault = true)]
     public class RazerLayerHandler : LayerHandler<RazerLayerHandlerProperties>
     {
-        private Color[] _keyboardColors;
-        private Color[] _mousepadColors;
+        private readonly Color[] _keyboardColors;
+        private readonly Color[] _mousepadColors;
         private Color _mouseColor;
         private string _currentAppExecutable;
-        private int _currentAppPid;
         private bool _isDumping;
+        private readonly EffectLayer _layer = new("Chroma Layer");
 
         public RazerLayerHandler()
         {
@@ -82,7 +82,6 @@ public RazerLayerHandler()
                 var appList = Global.razerSdkManager.GetDataProvider<RzAppListDataProvider>();
                 appList.Update();
                 _currentAppExecutable = appList.CurrentAppExecutable;
-                _currentAppPid = appList.CurrentAppPid;
             }
         }
 
@@ -93,7 +92,7 @@ protected override UserControl CreateControl()
 
         private void OnDataUpdated(object s, EventArgs e)
         {
-            if (!(s is AbstractDataProvider provider))
+            if (s is not AbstractDataProvider provider)
                 return;
 
             provider.Update();
@@ -118,7 +117,6 @@ private void OnDataUpdated(object s, EventArgs e)
             else if (provider is RzAppListDataProvider appList)
             {
                 _currentAppExecutable = appList.CurrentAppExecutable;
-                _currentAppPid = appList.CurrentAppPid;
             }
         }
 
@@ -150,33 +148,37 @@ public void DumpData(AbstractDataProvider provider)
         {
             var path = Path.Combine(Global.LogsDirectory, "RazerLayer");
             var filename = $"{provider.GetType().Name}_{Environment.TickCount}.bin";
-            using (var file = File.Open($@"{path}\{filename}", FileMode.Create)) {
-                var data = provider.Read();
-                file.Write(data, 0, data.Length);
-            }
+            using var file = File.Open($@"{path}\{filename}", FileMode.Create);
+            var data = provider.Read();
+            file.Write(data, 0, data.Length);
         }
 
+        private bool _empty = true;
         public override EffectLayer Render(IGameState gamestate)
         {
-            var layer = new EffectLayer();
-
             if (!IsCurrentAppValid())
-                return layer;
+            {
+                if (_empty) return _layer;
+                _layer.Clear();
+                _empty = true;
+                return _layer;
+            }
+            _empty = false;
 
             foreach (var key in (DeviceKeys[])Enum.GetValues(typeof(DeviceKeys)))
             {
-                if (!TryGetColor(key, out Color color))
+                if (!TryGetColor(key, out var color))
                     continue;
 
-                layer.Set(key, color);
+                _layer.Set(key, color);
             }
 
-            if (Properties.KeyCloneMap != null)
-                foreach (var target in Properties.KeyCloneMap)
-                    if(TryGetColor(target.Value, out var clr))
-                        layer.Set(target.Key, clr);
+            if (Properties.KeyCloneMap == null) return _layer;
+            foreach (var target in Properties.KeyCloneMap)
+                if(TryGetColor(target.Value, out var clr))
+                    _layer.Set(target.Key, clr);
 
-            return layer;
+            return _layer;
         }
 
         private bool TryGetColor(DeviceKeys key, out Color color)
@@ -199,7 +201,7 @@ private bool TryGetColor(DeviceKeys key, out Color color)
 
         private bool IsCurrentAppValid() 
             => !string.IsNullOrEmpty(_currentAppExecutable)
-            && string.Compare(_currentAppExecutable, "Aurora.exe", true) != 0;
+            && string.Compare(_currentAppExecutable, "Aurora.exe", StringComparison.OrdinalIgnoreCase) != 0;
 
         private Color PostProcessColor(Color color)
         {

Project-Aurora/Project-Aurora/packages.lock.json

@@ -76,13 +76,12 @@
       },
       "LibreHardwareMonitorLib": {
         "type": "Direct",
-        "requested": "[0.8.8, )",
-        "resolved": "0.8.8",
-        "contentHash": "KBPf4HJkGtubkFG8PL7Hr0E736CW4KLk6vqiC4aXY6quEd861/3tBo9h8VVvrL/q2r58xVq5zaoylIjqB5bqHA==",
+        "requested": "[0.8.9, )",
+        "resolved": "0.8.9",
+        "contentHash": "e0axsjvbjtQLdsuL25PwYgPsOAZjDORY+K+qgvtvUh8Rt4P/t+Xk0sDKRCHVcTlr/J6UGWKcd0dhmE5v25ujqQ==",
         "dependencies": {
           "HidSharp": "2.1.0",
-          "NvAPIWrapper.Net": "0.8.1.101",
-          "System.Management": "4.7.0"
+          "System.Management": "5.0.0"
         }
       },
       "MouseKeyHook": {
@@ -389,11 +388,6 @@
           "NAudio.Core": "2.0.0"
         }
       },
-      "NvAPIWrapper.Net": {
-        "type": "Transitive",
-        "resolved": "0.8.1.101",
-        "contentHash": "qX+5EQ32tCTqBkX/9BNfx7w4HPi18s3XhMubjbxSjPhYQ6pXWL+HGnaZEmGvFlOxvsQN+/6eV3x5bRo1CDY/mQ=="
-      },
       "SharpDX": {
         "type": "Transitive",
         "resolved": "4.0.1",