diff options
| author | wm4 <wm4@nowhere> | 2018-01-11 23:36:07 +0100 |
|---|---|---|
| committer | Kevin Mitchell <kevmitch@gmail.com> | 2018-01-13 03:26:45 -0800 |
| commit | 69d062ce373be7884bbde85e2b03f5c0f5789e05 (patch) | |
| tree | 855bdeff7e61df66c608944c29487099b5d1f903 /libmpv | |
| parent | 55993fde6df3fa38c47d2b08e6517053f4bc5ce4 (diff) | |
| download | mpv-69d062ce373be7884bbde85e2b03f5c0f5789e05.tar.bz2 mpv-69d062ce373be7884bbde85e2b03f5c0f5789e05.tar.xz | |
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.)
Diffstat (limited to 'libmpv')
| -rw-r--r-- | libmpv/client.h | 2 |
1 files changed, 2 insertions, 0 deletions
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 * --------------------- |