Rémi Denis-Courmont authored 2 years ago and Hugo Beauzée-Luyssen committed 2 years ago

Like on x86 (e.g. VLC_SSE), this macro enables the use of AltiVec for
just a given function. Thus the other functions in the same C module
can be compiled without AltiVec and run on non-AltiVec processors.

Rémi Denis-Courmont authored 2 years ago and Hugo Beauzée-Luyssen committed 2 years ago

This was added ostensibly to fix compilation of the run-time AltiVec
test which was removed in bc146294.
Unfortunately, it causes the compiler to emit AltiVec instructions,
such that the executable crashes if AltiVec is not available.

Regression from e48d6195.

Alexandre Janniaux authored 2 years ago and Steve Lhomme committed 2 years ago

It was missing from 514954e2.

This gets set explicitly when the video output is started, which always
results in the window being (re)enabled. Conversely, this gets
implicitly unset when the video output is stopped, which always results
in the window being disabled.

No functional changes.

This is no longer used.

This takes mouse coordinates translation out of the video output and
into the video window code, getting rid of one display lock dependency.

No functional changes.

Take only a struct vout_display_size rather than a full
struct vout_display_cfg_t. No functional changes.

Take only a struct vout_display_size rather than a full
struct vout_display_cfg_t. No functional changes.

Take only a struct vout_display_size rather than a full
struct vout_display_cfg_t. No functional changes.

Take only a struct vout_display_size rather than a full
struct vout_display_cfg_t. No functional changes.

This puts all the video size and placement properties back together as
vout_display_cfg_t originally was before other parameters were added.

No functional changes.

No functional changes.

Windows can send input events even if there is no display, so the
filter chains may need to be accessed even then. This protects the
creation and destruction of the chains with the existing filter lock,
providing memory safety.

Regression from 1ccf35ba.

Alexandre Janniaux authored 2 years ago and Hugo Beauzée-Luyssen committed 2 years ago

The opposite condition is already tested with an early return.

Alexandre Janniaux authored 2 years ago and Hugo Beauzée-Luyssen committed 2 years ago

`ret = ` is longer than `if ()` so is not needed to reduce the line
length, and the return value is not used except as early return.

Pierre Lamot authored 2 years ago and Jean-Baptiste Kempf committed 2 years ago

no functional changes