Code analysis fixes

arlm · Feb 25, 2019 · 4b72a05 · 4b72a05
1 parent 8e3b2a8
commit 4b72a05
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 17 deletions.
diff --git a/Chip8.Sharp/Chip8.Tests/Test.cs b/Chip8.Sharp/Chip8.Tests/Test.cs
@@ -2790,7 +2790,7 @@ public void Opcode_RND_Vx_byte()
                     break;
                 }
 
-                emptyMemory[testAddress] = unchecked((byte)(0xC0 | (0x00 & 0x0F)));
+                emptyMemory[testAddress] = 0xC0 | (0x00 & 0x0F);
                 emptyMemory[testAddress + 1] = unchecked((byte)value);
             }
 
@@ -3490,10 +3490,8 @@ private void Test_SKP_Vx(byte[] emptyMemory, int testAddress)
                             Assert.Throws<InvalidOperationException>(cpu.EmulateCycle);
                             continue;
                         }
-                        else
-                        {
-                            cpu.EmulateCycle();
-                        }
+
+                        cpu.EmulateCycle();
 
                         Assert.AreEqual(opcode, cpu.Opcode, $"Opcode shoud be 0x{opcode.ToString("X4", NumberFormatInfo.CurrentInfo)} - SKNP V{registerX.ToString("X1", NumberFormatInfo.CurrentInfo)}");
 
@@ -3571,10 +3569,8 @@ private void Test_SKNP_Vx(byte[] emptyMemory, int testAddress)
                             Assert.Throws<InvalidOperationException>(cpu.EmulateCycle);
                             continue;
                         }
-                        else
-                        {
-                            cpu.EmulateCycle();
-                        }
+
+                        cpu.EmulateCycle();
 
                         Assert.AreEqual(opcode, cpu.Opcode, $"Opcode shoud be 0x{opcode.ToString("X4", NumberFormatInfo.CurrentInfo)} - SKNP V{registerX.ToString("X1", NumberFormatInfo.CurrentInfo)}");
 

diff --git a/Chip8.Sharp/Chip8.WindowsForms/MainForm.cs b/Chip8.Sharp/Chip8.WindowsForms/MainForm.cs
@@ -20,10 +20,10 @@ public partial class MainForm : Form
         private readonly Bitmap screenImage;
         private readonly ICPU myChip8;
 
-        private bool quit = false;
-        private bool pause = false;
-        private bool debugKeys = false;
-        private bool debugPixels = false;
+        private bool quit;
+        private bool pause;
+        private bool debugKeys;
+        private bool debugPixels;
         private float zoom = 9.5f;
 
         // For timing..

diff --git a/Chip8.Sharp/Chip8.WindowsForms/PictureBoxWithInterpolationMode.cs b/Chip8.Sharp/Chip8.WindowsForms/PictureBoxWithInterpolationMode.cs
@@ -8,11 +8,11 @@ class PictureBoxWithInterpolationMode : PictureBox
         // http://stackoverflow.com/a/13484101/25124
         public InterpolationMode InterpolationMode { get; set; }
 
-        protected override void OnPaint(PaintEventArgs paintEventArgs)
+        protected override void OnPaint(PaintEventArgs pe)
         {
-            paintEventArgs.Graphics.PixelOffsetMode = PixelOffsetMode.Half;
-            paintEventArgs.Graphics.InterpolationMode = InterpolationMode;
-            base.OnPaint(paintEventArgs);
+            pe.Graphics.PixelOffsetMode = PixelOffsetMode.Half;
+            pe.Graphics.InterpolationMode = InterpolationMode;
+            base.OnPaint(pe);
         }
     }
 }