This reverts commit a19c8712.

Fixes both a memory leak and an alloc/free mismatch, and the module heap
is the most appropriate place to hold the variable long-term.

Stops a linker error of duplicate name when using upstream vkd3d.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55128Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Only done for DS format.
Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Add infrastructure to generate fake .PDB files.

Only done in DS format.

I'm not sure we care about JG anymore, DS format has been introduced
by MS in early 2000 to supersede JG, and is still mainstream today).
For the record, LLVM in its PDB handling doesn't support JG format.
Signed-off-by: Eric Pouech <epouech@codeweavers.com>

NtUserMessageCall(..., NtUserSendDriverMessage) does not ensure the
validity of the hwnd, so this check will incorrectly fail if the
last error was ERROR_INVALID_WINDOW_HANDLE by coincidence. The
notify_owner call immediately following this code will catch a
destroyed owner hwnd.

'if exists' takes a parameter which can be directory, directory/,
directory/. directory\ or directory\. for example, and should equate
to true if the directory exists. The syntax directory\ is explicitly
rejected by FindFirstFile and hence was not working - look for this
specific case, and if found append a '.'.

Follow-up commit of bc9d68bc

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55130#add_comment

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

- adjust hex/decimal print out
- use internal helpers
Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>

As it is done in exec_source. This matters as we actually have a check in
exprval_call that relies on this (because scope variable objects must never
be exposed).
Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>