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

PATH_MAX is used in FreeBSD-specific code in get_argv0_dir which
otherwise fails.

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com>

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

Signed-off-by: Bernhard Kölbl <besentv@gmail.com>