Re: [PATCH v8 0/2] MR4165: gdiplus: Fix transformation in GdipIsOutlineVisiblePathPoint

[Jeffrey Smith (＠whydoubt)]

On Fri Oct 27 07:26:25 2023 +0000, Bartosz Kosiorek wrote:
> I think we should completely remove World -> Device space conversion
> from GdipIsOutlineVisiblePathPoint, e.g. by passing transformation=NULL
> to `GdipWidenPath`: `GdipWidenPath(wide_path, pen, NULL, 1.0);`.
> As a result all conversion will be done in `GdipIsVisiblePathPoint`.
> It will simplify the code, and boost performance (no need to have
> redundant calculation in `GdipIsOutlineVisiblePathPoint`).``
> I could help with that after merging this PR (which make sure there will
> be no regression in `GdipIsOutlineVisiblePathPoint`).

I agree, keeping the path widening in world space sounds like a good solution.