From 9b86eaea7a52077624704d93afd0bb6bd84ce865 Mon Sep 17 00:00:00 2001
From: bbbradsmith <bbbradsmith@users.noreply.github.com>
Date: Fri, 11 Oct 2024 02:04:45 -0400
Subject: [PATCH] Fix palette display for 32bpp (was getting the wrong value
 due to signed 32-bit instead of unsigned)

---
 BinxelviewForm.cs | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/BinxelviewForm.cs b/BinxelviewForm.cs
index de660b4..cbc4199 100644
--- a/BinxelviewForm.cs
+++ b/BinxelviewForm.cs
@@ -739,7 +739,7 @@ unsafe int getPaletteRaw(long x)
             return autoPaletteRaw(x);
         }
 
-        Color getPalette(int x)
+        Color getPalette(long x)
         {
             if (preset.bpp <= PALETTE_BITS) return palette[x];
             int p = autoPaletteRaw(x);
@@ -1149,8 +1149,8 @@ void redrawPalette()
             {
                 for (int x = 0; x < PALETTE_DIM; ++x)
                 {
-                    int px = (x * (1 << bx)) / PALETTE_DIM;
-                    int py = (y * (1 << by)) / PALETTE_DIM;
+                    long px = ((long)x * (1 << bx)) / PALETTE_DIM;
+                    long py = ((long)y * (1 << by)) / PALETTE_DIM;
                     Color c = getPalette(px + (py * (1 << bx)));
                     palette_bmp.SetPixel(x, y, c);
                 }
@@ -1465,9 +1465,9 @@ private void paletteBox_MouseMove(object sender, MouseEventArgs e)
         {
             int bx = preset.bpp / 2;
             int by = preset.bpp - bx;
-            int px = (e.X * (1 << bx)) / PALETTE_DIM;
-            int py = (e.Y * (1 << by)) / PALETTE_DIM;
-            int index = px + (py * (1 << bx));
+            long px = ((long)e.X * (1 << bx)) / PALETTE_DIM;
+            long py = ((long)e.Y * (1 << by)) / PALETTE_DIM;
+            long index = px + (py * (1 << bx));
             Color c = getPalette(index);
             int ch = (c.R << 16) | (c.G << 8) | c.B;
             labelInfoPal.Text = String.Format("{0:D} = {1:D},{2:D},{3:D}\n{4:X2} = {5:X6}",(uint)index,c.R,c.G,c.B,(uint)index,ch);
@@ -1479,9 +1479,9 @@ private void paletteBox_MouseClick(object sender, MouseEventArgs e)
 
             int bx = preset.bpp / 2;
             int by = preset.bpp - bx;
-            int px = (e.X * (1 << bx)) / PALETTE_DIM;
-            int py = (e.Y * (1 << by)) / PALETTE_DIM;
-            int index = px + (py * (1 << bx));
+            long px = ((long)e.X * (1 << bx)) / PALETTE_DIM;
+            long py = ((long)e.Y * (1 << by)) / PALETTE_DIM;
+            long index = px + (py * (1 << bx));
             Color c = getPalette(index);
 
             ColorDialog d = new ColorDialog();
@@ -1491,7 +1491,7 @@ private void paletteBox_MouseClick(object sender, MouseEventArgs e)
             if (d.ShowDialog() == DialogResult.OK)
             {
                 palette_mode = PaletteMode.PALETTE_CUSTOM;
-                setPalette(index, d.Color.R, d.Color.G, d.Color.B);
+                setPalette((int)index, d.Color.R, d.Color.G, d.Color.B);
                 redrawPixels();
             }
             refreshPalette();
@@ -1707,7 +1707,7 @@ private void pixelsToPaletteToolStripMenuItem_Click(object sender, EventArgs e)
             {
                 long p = readPixel(read_pos);
                 if (p < 0) break;
-                Color c = getPalette((int)p); // can't modify the new palette because we read it here
+                Color c = getPalette(p); // can't modify the new palette because we read it here
                 new_pal[i] = c;
                 ++count;
                 read_pos += preset.pixel_stride_bit;