WinGui: Don't enable the Display Width control for Custom Anamorphic when KeepAR is checked. Implement Reduce function for PAR so that the behaviour matches the Linux GUI. Couple of Logic and ordering bug fixes.

1 files changed, 23 insertions, 0 deletions

diff --git a/win/CS/HandBrake.ApplicationServices/Interop/HandBrakeUtils.cs b/win/CS/HandBrake.ApplicationServices/Interop/HandBrakeUtils.cs
index 1acb411b0..b8fb24778 100644
--- a/win/CS/HandBrake.ApplicationServices/Interop/HandBrakeUtils.cs
+++ b/win/CS/HandBrake.ApplicationServices/Interop/HandBrakeUtils.cs
@@ -304,6 +304,29 @@ namespace HandBrake.ApplicationServices.Interop
             return JsonConvert.DeserializeObject<Geometry>(result);

         }

 

+        public static void Reduce(long den, long num, out long x, out long y)

+        {

+            // find the greatest common divisor of num & den by Euclid's algorithm

+            long n = num, d = den;

+            while (d > 0)

+            {

+                long t = d;

+                d = n % d;

+                n = t;

+            }

+

+            // at this point n is the gcd. if it's non-zero remove it from num

+            // and den. Otherwise just return the original values.

+            if (n > 0)

+            {

+                num /= n;

+                den /= n;

+            }

+

+            x = num;

+            y = den;

+        }

+

         /// <summary>

         /// Sends the message logged event to any registered listeners.

         /// </summary>