When introducing SAT>IP support, we added a fallback mechanism on
officially published channel lists in case the user neither specified
a custom list nor the used set-top box provided one.

With the end of the SAT>IP Alliance that was dissolved 3 years ago, the
fallback server is no longer available and there will be no replacement
so this feature was removed.

Fixes #28684

Fatih Uzunoğlu authored 9 months ago and Steve Lhomme committed 9 months ago

`CompositorDirectComposition::setupVoutWindow()` may be called before setup is
completed, yielding an assertion failure.

No need for a recursive call, we'll get the same values from the same chapter.

And if the Seek fails, we should know.

The inifial virtual segment is already checked for that.

No functional changes.

And simplify indentation.

This reverts commit 4ba01f28.

Let p_vsegment->Seek() decide what is valid or not.
The p_segment is actually the p_vsegment's current chapter's actual segment.
The current chapter segment may be NULL, but that doesn't mean we can't seek.
The current chapter segment cannot be NULL, per its constructor.

It's really the list of opened/known Matroska Segments.

We don't need to have the actual Segment.

Co-authored-by: Alaric Senat <alaric@videolabs.io>

Otherwise it's not spec-compliant.

Otherwise it's not spec-compliant.

It's only mandatory for ordered chapters.

So we can pass the proper type directly to each call.

The naming in Rust is bogus but it does means it's using UCRT [1].

[1] https://doc.rust-lang.org/nightly/rustc/platform-support/pc-windows-gnullvm.html

It may not have been available when adc4d846 was added.
But now it's a Tier 2 target [1]

[1] https://doc.rust-lang.org/nightly/rustc/platform-support/apple-darwin.html

Fixing wstring usage with NULL strings.

Thomas Guillem authored 9 months ago and Steve Lhomme committed 9 months ago

Even if for now, we have only one string (that was freed correctly).

Add the possibility to use strings in track options without leaking when
the option is overridden.

Thomas Guillem authored 9 months ago and Steve Lhomme committed 9 months ago

Because it uses local variables to the function.