GitLab

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

There's no point in passing a list of modules to load at the same time
there, and it's not supported.

If we have two valid input item pointers, then they are the same if
and only if the pointers are equal. No need to compare input item IDs.

Or they are not until we have 2 billions of them, and even then, the
sign is not relevant.

Without the legacy event handler, the distinction is superfluous.

Since there are always exactly one callback per event (the owner´s),
this is much simpler. This also removes unhandled error cases, and
provides more straightforward callback prototypes.

They had no listeners.

This was never triggered.

This is neither used nor documented.

The deinterlace module string is meaningless if deinterlacing is
disabled, and must be set if deinterlacing is enabled. So compare it
to NULL to indicate if deinterlacing is enabled or is not.

The original remains valid, there is no need to copy it.

Modules use higher-level wrappers, currently filter_chain_NewVideo().

Signed-off-by: Thomas Guillem <thomas@gllm.fr>

Signed-off-by: Thomas Guillem <thomas@gllm.fr>

default to 80°
Signed-off-by: Thomas Guillem <thomas@gllm.fr>

Signed-off-by: Thomas Guillem <thomas@gllm.fr>

Convert/Save dialog has had option to specify 'start time' but
no option for setting 'stop time' in the interface. The backend
'stop-time' option was already present. This patch simply adds
and enables the edit widget for end time in Qt interface.
Signed-off-by: Marvin Scholz <epirat07@gmail.com>

When bs_read_ue() returned 2^32-1, computing (val + 1) as 'int'
overflowed. With this patch, the conversion from unsigned to signed is
performed after the division by two, so that the absolute value range
is always within limits of the signed 32-bits integer type.

Also use fast types since the function are meant to be inlined.

not intuitive, but something was divided by numerator