WaveOutFlush( p_aout, true ) is always called from this function.

Since it always assumes it outputs unicode strings even though _UNICODE is not
defined.

It's already set by the PKG_PROG_PKG_CONFIG call below with a nicer order of
variables and more official documentation.

The mingw32/mingw64 pkg-config doesn't handle UNIX pathes

The pkg-config of these shells only understands windows/hybrid pathes, not UNIX
pathes. The pure Msys2 shell doesn't.

And they force a PKG_CONFIG_LIBDIR unless specifically set, even exporting the
variable is not sufficient.

We may use some forbidden APIs that are actually usable via WindowsStoreCompat

GetACP in the core, for example.

Many contribs use forbidden APIs that are mapped to valid APIs in the
winstorecompat. We make sure they map to these calls with the define and
linking with the matching library.

And no need to redefine snprintf for one call.

Reported-by: Awais Chishti <chishtiawais511@gmail.com>

Otherwise in multithread builds it often tries to copy the file before it's
even built.

They will be removed in the next release of libvpx:
https://chromium-review.googlesource.com/c/webm/libvpx/+/1003515

No functional changes.

No functional changes.

No functional changes

In the medium term, using C++ inheritance would probably be better.

If the decoder is valid for holding, then it is also valid for setting
a VLC object variable or adding a VLC object variable callback.
Otherwise, loading the decoder pointer or accessing the decoder is
undefined.

Either way, holding then releasing serves no purpose.

Note: input_GetEsObjects() should probably be removed in favor of the
new VBI input controls; but those are not currently exposed to input
managers.

This is now available in all supported versions (iOS 9, macOS 10.11).