There are cases in StructConversionFunction.definition where we will
generate copy code for extension struct members, without emitting the
definition of "in_ext" variable used in the copy code.

This issue is triggered by mismatches in the condition that guards the
generation of the "in_ext" definitions, and the condition(s) that govern
the generation of the member copy code (e.g., in
StructConversionFunction.member_needs_copy and
VkMember.needs_conversion).

In order to avoid such mismatches and the burden of having to keep the
conditions in sync, this commit generates the definition on demand, by
checking if it's actually needed by the member copy code.
Signed-off-by: Alexandros Frantzis <alexandros.frantzis@collabora.com>

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53932

Clang understands __attribute__ ((destructor)) syntax just like GCC. This avoids a
link failure when atexit() used in the fallback path is not available.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53930

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53930

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

The big win here is getting rid of the reimplementation of wcsdup.

Looks like a copy-paste error.

This fixes warnings with gcc 12.2.

(Copycat of 5fa25d00 about clear_rtv())
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53915Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53923