[PATCH v2 0/1] MR6662: win32u/zibdrv: Round get_arc_max_points() value to a multiple of 4.

[Aida Jonikienė]

This seems like a weird solution to me (so I'm not sure if the tests pass)

--
  v2: win32u/dibdrv: Round get_arc_max_points() value to a multiple of 4.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6662

[PATCH v2 1/1] win32u/dibdrv: Round get_arc_max_points() value to a multiple of 4.

[Aida Jonikienė]

From: Aida Jonikienė <aidas957(a)gmail.com>

Fixes: 5924ab4c515 ("win32u: Implement drawing transformed round rectangles.")
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57306
---
 dlls/win32u/dibdrv/graphics.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dlls/win32u/dibdrv/graphics.c b/dlls/win32u/dibdrv/graphics.c
index 414737fa8ca..84e541871fc 100644
--- a/dlls/win32u/dibdrv/graphics.c
+++ b/dlls/win32u/dibdrv/graphics.c
@@ -367,12 +367,13 @@ static unsigned int get_arc_max_points( DC *dc, const RECT *rect )
         {rect->right, rect->bottom},
         {rect->right, rect->top}
     };
-    unsigned int width, height;
+    unsigned int width, height, points;
 
     lp_to_dp( dc, pts, 4 );
     width = max( abs( pts[2].x - pts[0].x ), abs( pts[3].x - pts[1].x ));
     height = max( abs( pts[2].y - pts[0].y ), abs( pts[3].y - pts[1].y ));
-    return (width + height) * 3;
+    points = (width + height) * 3;
+    return (points % 4) == 0 ? points : (points + 3) & ~3);
 }
 
 /* backend for arc functions; extra_lines is -1 for ArcTo, 0 for Arc, 1 for Chord, 2 for Pie */
-- 
GitLab

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

[Aida Jonikienė]

On Sun Oct 13 16:52:49 2024 +0000, Aida Jonikienė wrote:
> changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/6662/diffs?diff_id=137876&start_sha=3b1a743d5de58c34aaf4a0bcdfde4af93a8b173d#7e258e82f92bfdcc9880f65005d937fdba90d7da_375_375)
> I accidentally committed a mistake (which should be corrected now)

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6662#note_84872

[Aida Jonikienė]

On Sun Oct 13 16:52:49 2024 +0000, Aida Jonikienė wrote:
> changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/6662/diffs?diff_id=137876&start_sha=3b1a743d5de58c34aaf4a0bcdfde4af93a8b173d#7e258e82f92bfdcc9880f65005d937fdba90d7da_376_375)
> I should really learn to use bitwise operators (anyway the change has been committed)

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6662#note_84873

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/win32u/dibdrv/graphics.c:
> {rect->right, rect->bottom},
>          {rect->right, rect->top}
>      };
> -    unsigned int width, height;
> +    unsigned int width, height, points;
>
> lp_to_dp( dc, pts, 4 );
>      width = max( abs( pts[2].x - pts[0].x ), abs( pts[3].x - pts[1].x ));
>      height = max( abs( pts[2].y - pts[0].y ), abs( pts[3].y - pts[1].y ));
> -    return (width + height) * 3;
> +    points = (width + height) * 3;
> +    return (points % 4) == 0 ? points : (points + 3) & ~3);

```suggestion:-0+0
    return (points + 3) & ~3;
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6662#note_84875