diff options
author | wm4 <wm4@nowhere> | 2019-11-30 01:07:24 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2019-11-30 01:07:24 +0100 |
commit | 0e376150a69c84220bed7f4d6a2a7525c888b09e (patch) | |
tree | 00c9c88a2f43568021252d557942cbccb5b213bb /player | |
parent | 464a045ed84035bf1c15ea2eb55d9efe1662f524 (diff) | |
download | mpv-0e376150a69c84220bed7f4d6a2a7525c888b09e.tar.bz2 mpv-0e376150a69c84220bed7f4d6a2a7525c888b09e.tar.xz |
command: remove property change notification from property dispatcher
Properties should handle this themselves. This basically sent redundant
notifications. I found only two places where the "proper" notification
was missing.
Diffstat (limited to 'player')
-rw-r--r-- | player/command.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/player/command.c b/player/command.c index b90edec6a4..cf85858f47 100644 --- a/player/command.c +++ b/player/command.c @@ -486,6 +486,7 @@ static int mp_property_stream_open_filename(void *ctx, struct m_property *prop, return M_PROPERTY_ERROR; mpctx->stream_open_filename = talloc_strdup(mpctx->stream_open_filename, *(char **)arg); + mp_notify_property(mpctx, prop->name); return M_PROPERTY_OK; } case M_PROPERTY_GET_TYPE: @@ -997,6 +998,7 @@ static int parse_node_chapters(struct MPContext *mpctx, } mp_notify(mpctx, MPV_EVENT_CHAPTER_CHANGE, NULL); + mp_notify_property(mpctx, "chapter-list"); return M_PROPERTY_OK; } @@ -3534,9 +3536,6 @@ int mp_property_do(const char *name, int action, void *val, struct command_ctx *cmd = ctx->command_ctx; int r = m_property_do(ctx->log, cmd->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; |