From 9bb46aa9442d3d655860f499d08a70d17fc4756d Mon Sep 17 00:00:00 2001 From: kd-11 Date: Wed, 1 Dec 2021 17:30:39 +0300 Subject: [PATCH] rsx: Simplify unconstrained aspect ratio conversion - There is a reason resolutions are defined by only a height variable. --- rpcs3/Emu/RSX/rsx_utils.cpp | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/rpcs3/Emu/RSX/rsx_utils.cpp b/rpcs3/Emu/RSX/rsx_utils.cpp index 5dc628c902..fd1ebd1a0a 100644 --- a/rpcs3/Emu/RSX/rsx_utils.cpp +++ b/rpcs3/Emu/RSX/rsx_utils.cpp @@ -183,20 +183,8 @@ namespace rsx return {}; } - const double old_aspect = 1. * image_dimensions.width / image_dimensions.height; - const double scaling_factor = get_aspect_ratio() / old_aspect; - size2u result{ image_dimensions.width, image_dimensions.height }; - - if (scaling_factor > 1.0) - { - result.width = static_cast(image_dimensions.width * scaling_factor); - } - else if (scaling_factor < 1.0) - { - result.height = static_cast(image_dimensions.height / scaling_factor); - } - - return result; + // Unconstrained aspect ratio conversion + return size2u{ static_cast(image_dimensions.height * get_aspect_ratio()), image_dimensions.height }; } areau avconf::aspect_convert_region(const size2u& image_dimensions, const size2u& output_dimensions) const