Lyndon Brown authored 5 years ago and Jean-Baptiste Kempf committed 3 years ago

the lowercase "auto" default option value did not match the uppercase
"Auto" in the choice list.

option values should typically be lowercase. this fixes the mistake with
the case of the choice value, whilst maintaining backwards compatibility
with existing CLI usage and from saved settings files. this was the
clear preference for fixing this resulting from the MR discussion.

Lyndon Brown authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

(no functional change)

 - better prepare for the next commit where we need to support both "Auto"
   and "auto".
 - avoid multiple `strdup()`.
 - avoid avoidable string comparisons.

Lyndon Brown authored 5 years ago and Jean-Baptiste Kempf committed 3 years ago

NULL does not map to an entry in the choice list.

also, mark the choice labels for translation.

Lyndon Brown authored 5 years ago and Jean-Baptiste Kempf committed 3 years ago

-1 was not in the choice list.

this adds the missing choice list entry corresponding to the default.
i've made a guess at the right label and explanation to use based upon
what seems to be happening with a -1 value in `stereo_to_mono()` and
from bits of git history relating to it.

Lyndon Brown authored 5 years ago and Jean-Baptiste Kempf committed 3 years ago

0 was not in the choice list.

it looks like 0 was interpreted by the code as the same as -1 via
`dvb_parse_transmit_mode()`, so not a big problem.

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

fix: #26236

Note that this now uses `unsigned char` rather than `char`, consistent
with the expectation for the `%c` format string specifier. This also
avoids using a signed value to index an array.

This paves the way to split private and public properties from
module_config_t.

Private core headers don't need this.

Steve Lhomme authored 3 years ago and Rémi Denis-Courmont committed 3 years ago

This would make the detection of a canceled thread vs normal run impossible.

Not implemented in posix/thread.c because we don't get the return code in our
code. We could do it with an indirect call.

Prince Gupta authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

fixup 96053261

They are not used anymore since it was moved to the interop_dxva2
module.

This will use extensions from wgl directly into interop_dxva2, instead
of using those loaded by wgl.

The only users of those extensions are interop modules, so only
interop_dxva2 for extension related to WGL integration.

By moving the loading of those extension to the interop, we can have
them working even when the OpenGL platform context is provided by the
vglmem (vgl.c) module callbacks, allowing hardware acceleration even
when using the libvlc OpenGL callbacks.

Filters would likely be using only OpenGL extensions that are already
available, but could be using the same mechanism in the worst-case
scenario.

Refs #25234, #25256

Prince Gupta authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

fixes associated compile time warning

Prince Gupta authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

fixes undefined sanitizers warnings -
../../modules/gui/qt/maininterface/main_interface.cpp:283:19: runtime
error: load of value 190, which is not a valid value for type 'bool'
.... etc

Hugo Beauzée-Luyssen authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

This will help automatically cancel outdated pipelines instead of having
to do so manually

Benjamin Arnaud authored 3 years ago and Hugo Beauzée-Luyssen committed 3 years ago

fix #26037

Romain Vimont authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

The orientation matrix was not initialized if orientation was
ORIENT_NORMAL.

The regression has been introduced by commit
01db27e1: since the matrix has been
reduced to 2x3, all the matrix components were set by the macro
MATRIX_SET(), so the prealable initialization to "identity" has been
removed.

However, the switch did not cover the case ORIENT_NORMAL, which had
nothing to do if the matrix was already initialized to "identity".

In practice, this caused a problem only on Android with hardware
acceleration enabled, because interop_android.c is the only interop
which does not apply a vflip to the input format (see commit
1fee4df0).

Alexandre Janniaux authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

Otherwise, it breaks when building in wsl with Cygwin installed:

    ~/vlc$ cygpath.exe -pm /
    C:/msys64/

There is no point in converting from and to the same path convention.

As briefly noted in 0a8a5333, that
changeset removed the confusing and messy limitations of mutex
assertions. Previously, it would only work if:
- checking for hold, not for non-hold,
- POSIX threads was used,
- LibVLC was built in debug mode,
- the mutex was initialised dynamically (not `VLC_STATIC_MUTEX`).

Version 2.15 of QtQuick.Window is not available in Qt 5.11

Fix regression introduced in e2c81a8a

Romain Vimont authored 3 years ago and Jean-Baptiste Kempf committed 3 years ago

A sampler now allows to handle chroma conversion from any OpenGL
picture provided by the filter. It can be the input picture directly
(most of the time), or any picture produced by the filter itself.

Therefore, make the sampler a simple helper for filters, independent of
the filter engine.