Removes the old VLCFSPanel, and references to the
source files from the makefiles, potfiles and
the Xcode project.

This will add a fallback mechanism that uses a
normal layer-backed NSView with a dark but still
slightly transparent background on macOS versions
that do not support NSVisualEffectView.

Additionally it will allow compilation with older
SDK's that have no NSVisualEffectView defined.
If compiled with such an SDK, support for
NSVisualEffectView is disabled completely, so
even when running the compiled VLC version
on 10.10 or higher it will use the old fallback
view, as the SDK it was compiled with had not the
necessary definitions.

This introduces a new fullscreen controller
which uses NSVisualEffectView for it's background
and the icons that are used throughout the rest of
the macOS user interface, hence making it more
consistent with the overall look of VLC.

Additionally the new fullscreen controller is
resizable and, as it already used to be, can be
move around on the screen.

The user interface is now described in a interface
builder file (xib) instead of doing everything in
code. This allows easy usage of autolayout, which
is heavily used to allow adaption of the
buttons/sliders depending on the panel width and
height.

This adds MACOSX_DEPLOYMENT_TARGET (which will be set to the value of
the MIN_OSX_VERSION variable) so contribs that only use xcodebuild
(like BGHUDKit) will use the correct deployment target.

Xcode 3 is ancient and Xcode 4 is available for OS X 10.7 (the oldest
OS X version that VLC supports) so this special case can be removed.

The function should provide interrupt DNS resolution, so a thread can
abort a DNS query (e.g. if network is down or really slow).

The stub is not interruptible. This makes no difference to the existing
uninterruptible (mis)behaviour.

guess from samples

according to samples, there always can be more than 8 bytes

refs HigashiNoEden8CCs.ts

Sebastian Ramacher authored 8 years ago and Rémi Denis-Courmont committed 8 years ago

This looks like a typo to me and should have been ALTIVEC_CFLAGS instead.

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Marvin Scholz authored 8 years ago and Rémi Denis-Courmont committed 8 years ago

libmicrodns does not work on macOS as bonjour is
used there, so it makes not much sense to build it
on macOS.

See https://github.com/videolabs/libmicrodns/issues/9



Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>

Filip Roséen authored 8 years ago and Thomas Guillem committed 8 years ago

The previous implementation inaccurately treated "p_sys->psz_base" as
if it was referring to whatever directory (uri-wise) that contains the
playlist we are currently processing.

Given that this is not the case, it would not handle relative paths
correctly.

These changes make sure that we use ProcessMRL to properly handle the
paths found within the playlist (the function handles both absolute
and relative ones).

fixes #17506

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

Filip Roséen authored 8 years ago and Thomas Guillem committed 8 years ago

These added checks are there to make sure that common mistakes when
parsing the port-specification of an URL are avoided.

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

Filip Roséen authored 8 years ago and Thomas Guillem committed 8 years ago

The previous implementation, somewhat unconditionally, made
assumptions about the internal state of the result of `vlc_UrlParse`,
meaning that it was inherently difficult to check if invalid URLs was
correctly rejected.

These changes simply asserts that vlc_UrlParse failed if all of the
arguments (used to verify the result) are NULL or 0.

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

Filip Roséen authored 8 years ago and Thomas Guillem committed 8 years ago

According to RFC3986, the port specifier does not have to contain
digits following the colon. This means that "http://example.com:" is a
valid URI, and that the relevant test should not be within the
"Invalid URIs"-group.

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

Filip Roséen authored 8 years ago and Thomas Guillem committed 8 years ago

Differences compared to the previous implementation:

 - accept URLs with empty port-specification (RFC3986, 3.2.3).

 - reject port-specification with leading sign (RFC3986 only
   allows *DIGIT).

refs #17555

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

Handle DTS magic sync code in only one place.

And rename b_dts_hd to b_substream. This boolean means that the dts frame is a
substream (extension of DTS-Core).

pi_samples is needed to check if the sync is followed by an other sync when the
WAV file has 0 padding.

If there is not enough room for it.

Filip Roséen authored 8 years ago and Pierre Ynard committed 8 years ago

Prior to this patch, the "info"-command could only be used to query
information related to the currently playing entity. These changes
allow the user to get information about other entities in the
playlist, based on the playlist-id.

fixes #17549

Signed-off-by: Pierre Ynard <linkfanel@yahoo.fr>

If buf[3] == 0xff, i_channels wraps to 0, and i_rate triggers SIGFPE.

With that options, plugins are *only* loaded from the cache. This
reduces the run time of libvlc_new() (about 2.5x on my system).

But new plugins won´t be picked up until the cache is regenerated.

Filip Roséen authored 8 years ago and Rémi Denis-Courmont committed 8 years ago

Given that module_InitDynamic can fail and return a NULL pointer to
signal the error, we cannot unconditionally set data-members of the
referred to vlc_plugin_t.

These changes fixes the issue by introducing an explicit branch
related to initialization.

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>