Re: [PATCH 0/1] MR1055: gdi32: Don't stackoverflow if substitute doesn't change anything

[Nikolay Sivov (＠nsivov)]

On Sat Oct 15 16:54:24 2022 +0000, Fabian Maurer wrote:
> Are you sure it works exactly like that in gdi32 as well? We don't seem
> to have tests for that, and I don't feel comfortable just removing the
> recursive call. I mean there's probably a reason it was done like this...
> Anyways, I just wanted to fix a stackoverflow, didn't expect to rewrite
> the logic for glyph lookup. Should I really change that with the
> superficial knowledge I have?

I don't know what you should do, I'm just pointing out why I think this change is conceptually not looking right.