Re: [PATCH v7 0/4] MR2864: gdiplus: Improve performance of GdipDrawImagePointsRect

[Esme Povirk (＠madewokherd)]

On Tue May 23 03:06:46 2023 +0000, Bartosz Kosiorek wrote:
> Previously `dst_dyn_data` was unused pointer defined here:
> dst_data = dst_dyn_data = heap_alloc_zero(sizeof(ARGB) * (dst_area.right
> - dst_area.left) * (dst_area.bottom - dst_area.top));     
>
> With current implementation, the pointer is used for iterate through
> memory. The increment is done now with `dst_dyn_data += sizeof(ARGB)`
> with `for` loop:
> for (x = dst_area.left; x < dst_area.right; x++, dst_dyn_data += sizeof(ARGB))
>
> To be able to not use `dst_stride` I have to switch `x` and `y` `for` loops.
> I have tested this implementation with application attached in description.

You shouldn't change `dst_dyn_data` because it gets freed later.