Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Even though that pragma is standard C99, MSVC doesn't understand it.
An issue has been opened, and this commit should be reverted when/if
they fix it:
https://connect.microsoft.com/VisualStudio/feedback/details/1009089/unhandled-standard-macros

WINBOOL is mingw specific

Fail if GetVlcChromaFormat return 0.
(It'll throw a HardwareAccelerationError)
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

- Fix possible infinite loop with sw rendering if GetOutput failed.
- Don't return a pic and don't release the block in case of error_state.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

This fix the random start fail of the module.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

This duplicates the 7-years older "zoom" option for no good reasons.

This makes sure that the "crop" variable always represents the current
crop settings.

The video output variables currently represent the wanted state, not
the actual state. Forcefully resetting them to the actual state with a
delay can create feedback loops. (Indeed some of the code paths had
already been commented out for that exact reason.)

This allows setting a value that is not explicitly listed a possible
choice, and also fixes a small race condition between VLC_VAR_DELCHOICE
and VLC_VAR_ADDCHOICE cals.

This allows adding choices as mere suggestions rather than strict set
of choices (just don´t set VLC_VAR_HASCHOICE in var_Create()).

All entries in a single list always have the same type anyway.

Signed-off-by: David Fuhrmann <dfuhrmann@videolan.org>

This solves long-standing conflicts with homebrew as well as the latest Xcode releases. Additionally, in most cases, all 3 items were built anyway.