Unfortunately, the C standard requires an 'unsigned char' value or EOF.
So we cannot blindly pass a 'char' value, which may be signed depending
on the architecture.

assert() is meant only for impossible errors (bugs), not for unlikely
errors. assert() does not check for errors in "release" builds.

There are no real functional changes Here. But it is cleaner not to
cast to block_sys_t if the block is not a block_sys_t block (i.e.
was not allocated by block_Alloc()).

Signed-off-by: François Revol <revol@free.fr>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Only allow static plugins if vlc binary is not built

The atom names are like FOURCC, but they are not fourcc

This reverts commit 6e54f3fd.

Conflicts:

	src/win32/thread.c

Trailing spaces, guid names consistency with MMS module, calloc, enum

Pointed-out-by: Michael A. Puls II

also simplify arm neon preprocessor checks

1tbs
remove hungarian prefixes

Audio buffers are destroyed downstream of the decoder, so this
function could never be used. In practice, the audio filters, the
audio output (or the encoder) already use block_Release() to delete
buffers allocated with decoder_NewAudioBuffer().

If "direct rendering" is ever implemented in audio output,
block_Release() custom buffer deletion can be used.