PVS-Studio stuff

Kawa-oneechan · Nov 9, 2023 · bd291c6 · bd291c6
1 parent 76e60a7
commit bd291c6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/SeqMaker/KawaTools.cs b/SeqMaker/KawaTools.cs
@@ -192,15 +192,12 @@ public void GetPixels(Bitmap from)
 				throw new FormatException("Input images can only be 320 by 200 pixels in size.");
 			}
 
-			if (palette != null)
+			for (var i = 0; i < victim.Palette.Entries.Length; i++)
 			{
-				for (var i = 0; i < victim.Palette.Entries.Length; i++)
-				{
-					var color = victim.Palette.Entries[i];
-					palette[i * 3 + 0] = color.R;
-					palette[i * 3 + 1] = color.G;
-					palette[i * 3 + 2] = color.B;
-				}
+				var color = victim.Palette.Entries[i];
+				palette[i * 3 + 0] = color.R;
+				palette[i * 3 + 1] = color.G;
+				palette[i * 3 + 2] = color.B;
 			}
 
 			var bitmapData = victim.LockBits(new Rectangle(0, 0, victim.Width, victim.Height), ImageLockMode.ReadOnly, victim.PixelFormat);

diff --git a/SeqMaker/Program.cs b/SeqMaker/Program.cs
@@ -262,7 +262,7 @@ static int Main(string[] args)
 					var frameWidth = frameRight - frameLeft;
 					var frameHeight = frameBottom - frameTop;
 
-					if (frameHeight + frameWidth == 0)
+					if (frameHeight <=0 || frameWidth <= 0)
 					{
 						frameWidth = 1;
 						frameHeight = 1;