Re: [PATCH v2 0/6] MR6916: winevulkan: Use vulkan object header for other objects.

[Rémi Bernon (＠rbernon)]

On Thu Nov 28 10:56:17 2024 +0000, Jacek Caban wrote:
> Why do you remove `wine_device_from_handle`? All those
> `CONTAINING_RECORD`s don't seem like an improvement. We could have both
> `wine_*` helpers (private in winevulkan) and `vulkan_*` (in `vulkan_driver.h`).

It didn't seem very useful to have intermediate helpers that simply wrap CONTAINING_RECORD and do the same thing under a different name? What benefit do they have over CONTAINING_RECORD? Could we improve CONTAINING_RECORD to do whatever it's missing instead of adding more helpers?