From d375cc304aec11f09ed685ffd2f2e7af0abf8910 Mon Sep 17 00:00:00 2001
From: wm4 <wm4@nowhere>
Date: Sat, 21 Mar 2020 22:09:07 +0100
Subject: client API, lua: unify event code further

Move some parts that can be generic to the client API code. It turns out
lua.c doesn't need anything special.

This adds the "id" field. I think this was actually missing from the
JSON IPC code (i.e. it's a very recent regression that is fixed with
this commit).
---
 player/lua.c | 60 ++++++++----------------------------------------------------
 1 file changed, 8 insertions(+), 52 deletions(-)

(limited to 'player/lua.c')

diff --git a/player/lua.c b/player/lua.c
index aa0b071eb5..76a0d69e52 100644
--- a/player/lua.c
+++ b/player/lua.c
@@ -505,62 +505,18 @@ static int script_wait_event(lua_State *L)
     mpv_event *event = mpv_wait_event(ctx->client, luaL_optnumber(L, 1, 1e20));
 
     lua_newtable(L); // event
-    lua_pushstring(L, mpv_event_name(event->event_id)); // event name
-    lua_setfield(L, -2, "event"); // event
 
-    if (event->reply_userdata) {
-        lua_pushnumber(L, event->reply_userdata);
-        lua_setfield(L, -2, "id");
-    }
+    struct mpv_node rn;
+    mpv_event_to_node(&rn, event);
 
-    if (event->error < 0) {
-        lua_pushstring(L, mpv_error_string(event->error)); // event err
-        lua_setfield(L, -2, "error"); // event
+    assert(rn.format == MPV_FORMAT_NODE_MAP);
+    mpv_node_list *list = rn.u.list;
+    for (int n = 0; n < list->num; n++) {
+        pushnode(L, &list->values[n]);
+        lua_setfield(L, -2, list->keys[n]);
     }
 
-    switch (event->event_id) {
-    case MPV_EVENT_PROPERTY_CHANGE: {
-        mpv_event_property *prop = event->data;
-        lua_pushstring(L, prop->name);
-        lua_setfield(L, -2, "name");
-        switch (prop->format) {
-        case MPV_FORMAT_NODE:
-            pushnode(L, prop->data);
-            break;
-        case MPV_FORMAT_DOUBLE:
-            lua_pushnumber(L, *(double *)prop->data);
-            break;
-        case MPV_FORMAT_FLAG:
-            lua_pushboolean(L, *(int *)prop->data);
-            break;
-        case MPV_FORMAT_STRING:
-            lua_pushstring(L, *(char **)prop->data);
-            break;
-        default:
-            lua_pushnil(L);
-        }
-        lua_setfield(L, -2, "data");
-        break;
-    }
-    case MPV_EVENT_COMMAND_REPLY: {
-        mpv_event_command *cmd = event->data;
-        pushnode(L, &cmd->result);
-        lua_setfield(L, -2, "result");
-        break;
-    }
-    default: ;
-        struct mpv_node rn;
-        mpv_event_to_node(&rn, event);
-
-        assert(rn.format == MPV_FORMAT_NODE_MAP);
-        mpv_node_list *list = rn.u.list;
-        for (int n = 0; n < list->num; n++) {
-            pushnode(L, &list->values[n]);
-            lua_setfield(L, -2, list->keys[n]);
-        }
-
-        mpv_free_node_contents(&rn);
-    }
+    mpv_free_node_contents(&rn);
 
     // return event
     return 1;
-- 
cgit v1.2.3