refactor(server): Nest if to prevent checking two times for the same …

…condition
schdck · Apr 14, 2020 · 04290e7 · 04290e7
1 parent 30744c8
commit 04290e7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/RemoteFlix.Base/RemoteFlixServer.cs b/RemoteFlix.Base/RemoteFlixServer.cs
@@ -213,12 +213,12 @@ private bool ExecuteCommand(BasePlayer player, PlayerCommand command)
             if (playerHandle == null)
             {
                 playerHandle = KeyboardSimulationHelper.GetForegroundWindow();
-            }
 
-            // If this also fails, there isn't much we can do
-            if(playerHandle == null || playerHandle.Value == IntPtr.Zero)
-            {
-                return false;
+                // If this also fails, there isn't much we can do
+                if (playerHandle.Value == IntPtr.Zero)
+                {
+                    return false;
+                }
             }
 
             KeyboardSimulationHelper.SendKeys(playerHandle.Value, command.ActionShortcut);

diff --git a/RemoteFlix.UI.Desktop/ViewModel/MainViewModel.cs b/RemoteFlix.UI.Desktop/ViewModel/MainViewModel.cs
@@ -112,7 +112,7 @@ private void SetupEnvironment()
         private void LogReceived(object sender, NotifyCollectionChangedEventArgs e)
         {
             // It is safe to do this since there's only one method to modify the collection and it 
-            // adds (one) log. So every time the collection changes it is an addition of one item.
+            // adds *one* log. So every time the collection changes it is an addition of one item.
             var log = e.NewItems[0] as Log;
 
             Application.Current.Dispatcher.Invoke(() => Logs.Add(log));