[PATCH 0/1] MR7309: windowscodecs: Simplify a bit FlipRotator_CopyPixels() implementation.

[Dmitry Timoshkov (＠dmitry)]

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7309

[PATCH 1/1] windowscodecs: Simplify a bit FlipRotator_CopyPixels() implementation.

[Dmitry Timoshkov]

From: Dmitry Timoshkov <dmitry(a)baikal.ru>

Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru>
---
 dlls/windowscodecs/fliprotate.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/dlls/windowscodecs/fliprotate.c b/dlls/windowscodecs/fliprotate.c
index 35cfde34280..6c7ecb963da 100644
--- a/dlls/windowscodecs/fliprotate.c
+++ b/dlls/windowscodecs/fliprotate.c
@@ -156,8 +156,7 @@ static HRESULT WINAPI FlipRotator_CopyPixels(IWICBitmapFlipRotator *iface,
 {
     FlipRotator *This = impl_from_IWICBitmapFlipRotator(iface);
     HRESULT hr;
-    UINT y, width, height;
-    UINT srcy, srcwidth, srcheight;
+    UINT y, srcy, width, height;
     WICRect rc;
     WICRect rect;
 
@@ -165,9 +164,6 @@ static HRESULT WINAPI FlipRotator_CopyPixels(IWICBitmapFlipRotator *iface,
 
     if (!This->source) return WINCODEC_ERR_WRONGSTATE;
 
-    hr = IWICBitmapSource_GetSize(This->source, &srcwidth, &srcheight);
-    if (FAILED(hr)) return hr;
-
     hr = IWICBitmapFlipRotator_GetSize(iface, &width, &height);
     if (FAILED(hr)) return hr;
 
@@ -226,7 +222,7 @@ static HRESULT WINAPI FlipRotator_CopyPixels(IWICBitmapFlipRotator *iface,
     for (y=prc->Y; y - prc->Y < prc->Height; y++)
     {
         if (This->flip_y)
-            srcy = srcheight - 1 - y;
+            srcy = height - 1 - y;
         else
             srcy = y;
 
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7309

[PATCH 0/1] MR7309: windowscodecs: Simplify a bit FlipRotator_CopyPixels() implementation. - approved

[Esme Povirk (＠madewokherd)]

This merge request was approved by Esme Povirk.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7309