Use C functions instead of an ObjC singleton for string operations and move all string related operators to a single header/implementation

This is little more than a stub and will serve as a base for the window displaying the new library

The opaque value is for the mouse_event callback.

It does the same as CommonControl().

Re-use an existing string instead of creating a new one and hide the path action item if there is no action to perform

macosx/coreinteraction: split apple remote, media key and video filter handling as those don't belong there

No functional changes

Asking NSFileManager for the mounted devices can take very long (>6s) when the devices are slow to respond, so let's do that on the background thread and don't delay display of the open panel.

libplacebo won't find it

Asynchronous requests from the user interface should still be queued
without blocking, even if the video output is held by other processing.

For that, we need to release the mutex between Hold() and Release().

There is a mixture of subpicture-related calls, with some running on
the calling thread, and others running on the video output thread
control loop.

The later may be delayed, depending on other video output thread
processing, sleeping and pending control requests. This leads to a
race condition, and potentially inconsistent ordering of calls on the
SPU instance.

This patch runs vout_FlushSubpictureChannel() directly on the calling
thread and adds a missing check for nonexistent/destroyed SPU.

There is a mixture of subpicture-related calls, with some running on
the calling thread, and others running on the video output thread
control loop.

The later may be delayed, depending on other video output thread
processing, sleeping and pending control requests. This leads to a
race condition, and potentially inconsistent ordering of calls on the
SPU instance.

This patch runs vout_SubPicture() directly on the calling thread and
adds a missing check for nonexistent/destroyed SPU.

Romain Vimont authored 6 years ago and Thomas Guillem committed 6 years ago

If a media source was already created in _GetMediaSource(), its refcount
was not incremented.

Reported-by: Hugo Beauzée-Luyssen <hugo@beauzee.fr>
Signed-off-by: Thomas Guillem <thomas@gllm.fr>

On my system it doesn't find any matching format.

And build using cmake rather their manually edited makefiles

In recent wglew.h it relies on the DLL export mode defined in glew.h

... leveraging the removal of callbacks from previous changeset.

Once the callbacks are deleted, we are assured that they will no longer
be triggered. Then we can safely destroy resources without the risk of
causing use-after-free if a user interface sets a variable.