1
0
mirror of https://github.com/mpv-player/mpv synced 2024-12-24 15:52:25 +00:00

command: log property set calls

And remove the same thing from the client API code.

The command.c code has to deal with many specialized M_PROPERTY_SET_*
actions, and we bother with a subset only.
This commit is contained in:
wm4 2016-04-15 13:20:05 +02:00
parent 177f5d9e9c
commit fbb5da0010
2 changed files with 17 additions and 8 deletions

View File

@ -1085,14 +1085,6 @@ static void setproperty_fn(void *arg)
req->status = translate_property_error(err);
if (mp_msg_test(req->mpctx->log, MSGL_V)) {
struct m_option ot = {.type = &m_option_type_node};
char *t = m_option_print(&ot, node);
MP_VERBOSE(req->mpctx, "Set property: %s=%s -> %d\n",
req->name, t ? t : "?", err);
talloc_free(t);
}
if (req->reply_ctx) {
status_reply(req->reply_ctx, MPV_EVENT_SET_PROPERTY_REPLY,
req->userdata, req->status);

View File

@ -3904,6 +3904,23 @@ int mp_property_do(const char *name, int action, void *val,
int r = m_property_do(ctx->log, mp_properties, name, action, val, ctx);
if (r == M_PROPERTY_OK && is_property_set(action, val))
mp_notify_property(ctx, (char *)name);
if (mp_msg_test(ctx->log, MSGL_V) && is_property_set(action, val)) {
struct m_option ot = {0};
void *data = val;
switch (action) {
case M_PROPERTY_SET_NODE:
ot.type = &m_option_type_node;
break;
case M_PROPERTY_SET_STRING:
ot.type = &m_option_type_string;
data = &val;
break;
}
char *t = ot.type ? m_option_print(&ot, data) : NULL;
MP_VERBOSE(ctx, "Set property: %s%s%s -> %d\n",
name, t ? "=" : "", t ? t : "", r);
talloc_free(t);
}
return r;
}