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