On Wed May 15 12:16:18 2024 +0000, eric pouech wrote:
> [err](/uploads/6c39c40a566da58e25661ae18e455f1c/err)
Yes I was thinking about something like you have attached first but I didn't like few things:
- it requires adding new flag
- it's a special marker that is only valid in template, it can't be used as datatype in other places
- your code is also not eating final '@' correctly but it can be addressed easily
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5646#note_70520
--
v2: d3dx9: Fix destination rectangles passed from D3DXLoadSurfaceFromMemory() to d3dx_load_image_from_memory().
d3dx9: Only do direct copies of full blocks for compressed formats.
d3dx9/tests: Add a new compressed surface loading test.
d3dx9/tests: Add a test for negative values in the source rectangle passed to D3DXLoadSurfaceFromMemory.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5644
On Wed May 15 08:07:21 2024 +0000, eric pouech wrote:
> that test looks very strange at that place while we already have
> handling of other types starting with "$$" in demangle_datatype.
> it calls for having a proper identification about end of list (didn't
> include the function arg one, but we could well do it)
> I think we could rather use something like the attached patch.
> [err](/uploads/8c2f1adecffbfe57c3d3bf924bf1755a/
I can't see the patch, could you please fix the link?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5646#note_70505
eric pouech (@epo) commented about dlls/msvcrt/undname.c:
> sym->current++;
> break;
> }
> + /* Handle empty list in variadic template */
> + if (!z_term && sym->current[0] == '$' && sym->current[1] == '$' && sym->current[2] == 'V')
that test looks very strange at that place while we already have handling of other types starting with "$$" in demangle_datatype.
it calls for having a proper identification about end of list (didn't include the function arg one, but we could well do it)
I think we could rather use something like the attached patch.
[err](/uploads/8c2f1adecffbfe57c3d3bf924bf1755a/
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5646#note_70497