Okay!

3 files changed, 14 insertions, 12 deletions

diff --git a/libgui/gfx.c b/libgui/gfx.c
index d3a66f2..a6e23e6 100644
--- a/libgui/gfx.c
+++ b/libgui/gfx.c
@@ -100,8 +100,8 @@ static void draw_rectangle(struct context *ctx, int x1, int y1, int x2, int y2,
 struct context *gfx_new_ctx(struct context *ctx)
 {
 	struct message msg = { 0 };
-	msg_send(pidof(WM_PATH), GFX_NEW_CONTEXT, ctx);
-	msg_receive(&msg);
+	assert(msg_send(pidof(WM_PATH), GFX_NEW_CONTEXT, ctx) > 0);
+	assert(msg_receive(&msg) > 0);
 	memcpy(ctx, msg.data, sizeof(*ctx));
 	return ctx;
 }
diff --git a/libgui/gui.c b/libgui/gui.c
index 052a07b..8eb7ce9 100644
--- a/libgui/gui.c
+++ b/libgui/gui.c
@@ -23,8 +23,7 @@ static struct window windows[MAX_WINDOWS] = { 0 };
 
 static struct window *new_window(const char *title, int x, int y, u32 width, u32 height, int flags)
 {
-	if (window_count + 1 >= MAX_WINDOWS)
-		return NULL;
+	assert(window_count + 1 <= MAX_WINDOWS);
 
 	struct window *win = &windows[window_count + 1];
 	win->ctx = malloc(sizeof(*win->ctx));
@@ -200,7 +199,6 @@ static int absolute_y_off(struct element *elem)
 
 struct context *gui_get_context(int x, int y, u32 width, u32 height)
 {
-	log("GET CONTEXT: %dx%d\n", width, height);
 	struct context *ctx = malloc(sizeof(*ctx));
 	ctx->pid = getpid();
 	ctx->x = x;
@@ -535,10 +533,7 @@ void gui_event_loop(struct element *container)
 	struct message msg = { 0 };
 	struct element *focused = NULL;
 	while (1) {
-		/* if (!msg_receive(&msg)) { */
-		yield();
-		continue;
-		/* } */
+		assert(msg_receive(&msg) > 0);
 
 		switch (msg.type) {
 		case GUI_KILL: {
@@ -591,6 +586,9 @@ void gui_event_loop(struct element *container)
 			gui_sync_window(container->window_id);
 			break;
 		}
+		default: {
+			log("Unknown GUI request %d\n", msg.type);
+		}
 		}
 	}
 
diff --git a/libgui/msg.c b/libgui/msg.c
index bf3d28c..4576a5f 100644
--- a/libgui/msg.c
+++ b/libgui/msg.c
@@ -5,19 +5,23 @@
 #include <print.h>
 #include <sys.h>
 
+static struct message msg_buf = { 0 };
+
 int msg_send(u32 pid, enum message_type type, void *data)
 {
 	assert((signed)pid != -1);
 	char path[32] = { 0 };
 	sprintf(path, "/proc/%d/msg", pid);
-	struct message msg = { .magic = MSG_MAGIC, .src = getpid(), .type = type, .data = data };
-	return write(path, &msg, 0, sizeof(msg));
+	msg_buf.magic = MSG_MAGIC;
+	msg_buf.src = getpid();
+	msg_buf.type = type;
+	msg_buf.data = data;
+	return write(path, &msg_buf, 0, sizeof(msg_buf));
 }
 
 int msg_receive(struct message *msg)
 {
 	int ret = read("/proc/self/msg", msg, 0, sizeof(*msg));
-	assert(msg->magic == MSG_MAGIC); // TODO: Remove? (Many >0 checks are needed)
 	if (msg->magic == MSG_MAGIC && ret == sizeof(*msg))
 		return ret;
 	else