RepoMirrors/mpv
1
0
Fork 0
mirror of https://github.com/mpv-player/mpv synced 2025-02-17 21:27:08 +00:00
Code Issues Releases Wiki Activity

js: reimplement subprocess_detached using the run command (match 7f91e268)

Browse Source
This commit is contained in:
Avi Halachmi (:avih) 2019-03-12 00:47:29 +02:00
parent fea39b5a6b
commit ad0a525f20
2 changed files with 5 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
player/javascript.c
View File

@ -37,7 +37,6 @@
#include "input/input.h"
#include "options/path.h"
#include "misc/bstr.h"
#include "osdep/subprocess.h"
#include "osdep/timer.h"
#include "osdep/threads.h"
#include "osdep/getpid.h"
@ -920,36 +919,6 @@ static void script_get_user_path(js_State *J, void *af)
js_pushstring(J, mp_get_user_path(af, jctx(J)->mpctx->global, path));
}
// args: client invocation args object
static void script_subprocess_detached(js_State *J, void *af)
{
struct script_ctx *ctx = jctx(J);
if (!js_isobject(J, 1))
js_error(J, "argument must be an object");
js_getproperty(J, 1, "args"); // args
int num_args = js_getlength(J, -1);
if (!num_args) // not using js_isarray to also accept array-like objects
js_error(J, "args must be an non-empty array");
char *args[256];
if (num_args > MP_ARRAY_SIZE(args) - 1) // last needs to be NULL
js_error(J, "too many arguments");
if (num_args < 1)
js_error(J, "program name missing");
for (int n = 0; n < num_args; n++) {
js_getindex(J, -1, n);
if (js_isundefined(J, -1))
js_error(J, "program arguments must be strings");
args[n] = talloc_strdup(af, js_tostring(J, -1));
js_pop(J, 1); // args
}
args[num_args] = NULL;
mp_subprocess_detached(ctx->log, args);
push_success(J);
}
// args: none
static void script_getpid(js_State *J)
{
@ -1296,7 +1265,6 @@ static const struct fn_entry utils_fns[] = {
FN_ENTRY(split_path, 1),
AF_ENTRY(join_path, 2),
AF_ENTRY(get_user_path, 1),
AF_ENTRY(subprocess_detached, 1),
FN_ENTRY(getpid, 0),
FN_ENTRY(read_file, 2),

5
player/javascript/defaults.js
View File

@ -552,6 +552,11 @@ mp.utils.subprocess = function subprocess(t) {
return rv;
}
mp.utils.subprocess_detached = function subprocess_detached(t) {
return mp.commandv.apply(null, ["run"].concat(t.args));
}
// ----- dump: like print, but expands objects/arrays recursively -----
function replacer(k, v) {
var t = typeof v;