From 69d062ce373be7884bbde85e2b03f5c0f5789e05 Mon Sep 17 00:00:00 2001 From: wm4 Date: Thu, 11 Jan 2018 23:36:07 +0100 Subject: client API: remove ytdl=no default With the recent changes to the script it does not incur a startup delay by default due to starting youtube-dl and waiting for it. This was the main reason for making libmpv have a different default. Starting sub processes from a library can still be a bit fishy, but I think it's ok. Still mention it in the libmpv header. There were already other cases where libmpv would start its own processes, such as the X11 backend calling xdg-screensaver. (The reason why this is fishy is because UNIX process management sucks: SIGCHLD and the wait() syscall make sub processes non-transparent and could potentially introduce conflicts with code trying to use them.) --- libmpv/client.h | 2 ++ 1 file changed, 2 insertions(+) (limited to 'libmpv') diff --git a/libmpv/client.h b/libmpv/client.h index 91a8e81cbb..41bf4e0c5c 100644 --- a/libmpv/client.h +++ b/libmpv/client.h @@ -134,6 +134,8 @@ extern "C" { * - The FPU precision must be set at least to double precision. * - On Windows, mpv will call timeBeginPeriod(1). * - On memory exhaustion, mpv will kill the process. + * - In certain cases, mpv may start sub processes (such as with the ytdl + * wrapper script). * * Encoding of filenames * --------------------- -- cgit v1.2.3