File melonds_PR1686.patch of Package melonds

Overview Repositories Revisions Requests Users Attributes Meta

File melonds_PR1686.patch of Package melonds (Revision e15fc2138356a4ee37e409b6969fddc6)

Currently displaying revision e15fc2138356a4ee37e409b6969fddc6 , Show latest

From e9eb5a2f718d971954d5c41abdad62cebaa31987 Mon Sep 17 00:00:00 2001
From: Jaklyy <102590697+Jaklyy@users.noreply.github.com>
Date: Sun, 21 May 2023 16:22:13 -0400
Subject: [PATCH 1/3] Fix Up Y Interp Inputs

---
 src/GPU3D_Soft.h | 27 ++++++---------------------
 1 file changed, 6 insertions(+), 21 deletions(-)

diff --git a/src/GPU3D_Soft.h b/src/GPU3D_Soft.h
index c792f131f..408646a52 100644
--- a/src/GPU3D_Soft.h
+++ b/src/GPU3D_Soft.h
@@ -326,20 +326,12 @@ class SoftRenderer : public Renderer3D
 
             s32 x = XVal();
 
-            if (XMajor)
-            {
-                if (side) Interp.Setup(x0-1, x1-1, w0, w1); // checkme
-                else      Interp.Setup(x0, x1, w0, w1);
-                Interp.SetX(x);
+            int interpoffset = (Increment >= 0x40000) && (side ^ Negative);
+            Interp.Setup(y0-interpoffset, y1-interpoffset, w0, w1);
+            Interp.SetX(y);
 
-                // used for calculating AA coverage
-                xcov_incr = (ylen << 10) / xlen;
-            }
-            else
-            {
-                Interp.Setup(y0, y1, w0, w1);
-                Interp.SetX(y);
-            }
+            // used for calculating AA coverage
+            if (XMajor) xcov_incr = (ylen << 10) / xlen;
 
             return x;
         }
@@ -350,14 +342,7 @@ class SoftRenderer : public Renderer3D
             y++;
 
             s32 x = XVal();
-            if (XMajor)
-            {
-                Interp.SetX(x);
-            }
-            else
-            {
-                Interp.SetX(y);
-            }
+            Interp.SetX(y);
             return x;
         }

From 528930358efadca2627adc128463c13bca80bedd Mon Sep 17 00:00:00 2001
From: Jaklyy <102590697+Jaklyy@users.noreply.github.com>
Date: Mon, 22 May 2023 01:07:05 -0400
Subject: [PATCH 2/3] Change Linear Interp Formula

Fixes a handful of pixels.
Still not perfect.
---
 src/GPU3D_Soft.h | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/GPU3D_Soft.h b/src/GPU3D_Soft.h
index 408646a52..ff1053f3f 100644
--- a/src/GPU3D_Soft.h
+++ b/src/GPU3D_Soft.h
@@ -156,11 +156,10 @@ class SoftRenderer : public Renderer3D
             else
             {
                 // linear interpolation
-                // checkme: the rounding bias there (3<<24) is a guess
                 if (y0 < y1)
-                    return y0 + ((((s64)(y1-y0) * x * xrecip) + (3<<24)) >> 30);
+                    return y0 + ((s64)(y1-y0) * x / xdiff);
                 else
-                    return y1 + ((((s64)(y0-y1) * (xdiff-x) * xrecip) + (3<<24)) >> 30);
+                    return y1 + (((s64)(y0-y1) * (xdiff - x)) / xdiff);
             }
         }

From 0949bba1e4f836bd92d72003fba93cf5ef4d3be6 Mon Sep 17 00:00:00 2001
From: Jaklyy <102590697+Jaklyy@users.noreply.github.com>
Date: Wed, 24 May 2023 21:41:33 -0400
Subject: [PATCH 3/3] Cleanup

remove some unnecessary code and parentheses
---
 src/GPU3D_Soft.h | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/GPU3D_Soft.h b/src/GPU3D_Soft.h
index ff1053f3f..25a2d6cb1 100644
--- a/src/GPU3D_Soft.h
+++ b/src/GPU3D_Soft.h
@@ -78,13 +78,12 @@ class SoftRenderer : public Renderer3D
             this->x1 = x1;
             this->xdiff = x1 - x0;
 
-            // calculate reciprocals for linear mode and Z interpolation
+            // calculate reciprocal for Z interpolation
             // TODO eventually: use a faster reciprocal function?
             if (this->xdiff != 0)
-                this->xrecip = (1<<30) / this->xdiff;
+                this->xrecip_z = (1<<22) / this->xdiff;
             else
-                this->xrecip = 0;
-            this->xrecip_z = this->xrecip >> 8;
+                this->xrecip_z = 0;
 
             // linear mode is used if both W values are equal and have
             // low-order bits cleared (0-6 along X, 1-6 along Y)
@@ -157,9 +156,9 @@ class SoftRenderer : public Renderer3D
             {
                 // linear interpolation
                 if (y0 < y1)
-                    return y0 + ((s64)(y1-y0) * x / xdiff);
+                    return y0 + (s64)(y1-y0) * x / xdiff;
                 else
-                    return y1 + (((s64)(y0-y1) * (xdiff - x)) / xdiff);
+                    return y1 + (s64)(y0-y1) * (xdiff - x) / xdiff;
             }
         }
 
@@ -219,7 +218,7 @@ class SoftRenderer : public Renderer3D
         int shift;
         bool linear;
 
-        s32 xrecip, xrecip_z;
+        s32 xrecip_z;
         s32 w0n, w0d, w1d;
 
         u32 yfactor;

Places

File melonds_PR1686.patch of Package melonds (Revision e15fc2138356a4ee37e409b6969fddc6)

Places