Try to apply consistent naming:
- file refers to (PDB) file
- stream refers to a stream/file inside the PDB stream at MSF level
(we were also using file for the later, which isn't very simple to
follow).
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

We were using either _size or _len depending on which one.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

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

No longer calling it sometimes 'page'.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

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

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

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

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

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

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

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

0108:trace:secur32:InitializeSecurityContextA 25528220 00000000 (null) 0x0008c13c 0 16 ...
0108:trace:secur32:schan_InitializeSecurityContextA 25528220 00000000 (null) 573756 0 16 ...
0108:trace:secur32:schan_InitializeSecurityContextW 25528220 00000000 (null) 0x0008c13c 0 16 ...

These have cdecl convention, not stdcall.

Originally meant to address the fact that mult_alaw_sample() and
mult_ulaw_sample() were exported without being hidden, but this file is only
used from pulse.c, and is relatively quite simple, so just make them static
and use the preprocessor instead of linking.

Instead of passing them from the stack. The async operation is executed
in a thread pool, which might Release its reference asynchronously and
call a stale function pointer if it is sequenced after the stack
invalidation.

This informs the compiler that no code following an assertion failure
will execute (unless NDEBUG is defined), which increases the accuracy of
compiler warnings and static analyses.

For assert.h, put DECLSPEC_NORETURN before the return type in the
function declaration to remain consistent with other MSVCRT function
declarations.

This informs the compiler that no code following a RtlRaiseStatus() call
will execute, which increases the accuracy of compiler warnings and
static analyses.

It shows the same behavior as the one marked by commit 2d19baa6.

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

Make sure we pull in the definition of X11DRV_XRandR_Init(), in particular because it has DECLSPEC_HIDDEN.

According to the GCC documentation,[1] __attribute__((malloc)) may not
be used on functions that return memory that contains one or more valid
pointers.

[1] https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html