From 176cc7b9c1d07e3a3cb6c755cb8d213a5fa45009 Mon Sep 17 00:00:00 2001
From: Bill Currie <bill@taniwha.org>
Date: Sat, 21 Mar 2020 21:05:34 +0900
Subject: [PATCH] [qwaq] Make some improvements to mouse handling

This doesn't fix the problem of lost events: that seems to be inside
ncurses. I've done some investigations, and it seems xterm sends
separate events for motion and pre/release (which have current coords),
in both 1003 and 1006 modes. No idea what ncurses is doing (does it even
handle 1003 properly?), and it requires the use of xterm-1006 for it to
use 1006 mode (which is nice in that it disambiguates button releases
and allows for huge terminals (not that I would use such normally)).
Guess I've got some side-work cut out for me :P
---
 ruamoko/qwaq/event.h       | 10 +++++-
 ruamoko/qwaq/qwaq-curses.c | 65 ++++++++++++++++++++++++++++++--------
 2 files changed, 60 insertions(+), 15 deletions(-)

diff --git a/ruamoko/qwaq/event.h b/ruamoko/qwaq/event.h
index 03f479fdc..56327c42d 100644
--- a/ruamoko/qwaq/event.h
+++ b/ruamoko/qwaq/event.h
@@ -9,6 +9,14 @@ typedef enum {
 	qe_mouseauto = 0x0010,
 } qwaq_mouse_event;
 
+typedef enum {
+	qe_mouse1    = 0x0001,
+	qe_mouse2    = 0x0002,
+	qe_mouse3    = 0x0004,
+	qe_mouse4    = 0x0008,
+	qe_mouse5    = 0x0010,
+} qwaq_button;
+
 typedef enum {
 	qe_keydown   = 0x0020,
 } qwaq_key_event;
@@ -37,7 +45,7 @@ typedef enum {
 
 typedef struct qwaq_mevent_s {
 	int         x, y;
-	int         buttons;
+	qwaq_button buttons;	// current button state
 	int         click;
 } qwaq_mevent_t;
 
diff --git a/ruamoko/qwaq/qwaq-curses.c b/ruamoko/qwaq/qwaq-curses.c
index 9cc17ab6c..6ab3f6154 100644
--- a/ruamoko/qwaq/qwaq-curses.c
+++ b/ruamoko/qwaq/qwaq-curses.c
@@ -1,12 +1,12 @@
 /*
-	#FILENAME#
+	qwaq-curses.c
 
-	#DESCRIPTION#
+	Ruamoko ncurses wrapper using threading
 
-	Copyright (C) 2001 #AUTHOR#
+	Copyright (C) 2020 Bill Currie <bill@taniwha.org>
 
-	Author: #AUTHOR#
-	Date: #DATE#
+	Author: Bill Currie <bill@taniwha.org>
+	Date: 2020/03/01
 
 	This program is free software; you can redistribute it and/or
 	modify it under the terms of the GNU General Public License
@@ -54,8 +54,10 @@
 
 #define always_inline inline __attribute__((__always_inline__))
 #define QUEUE_SIZE 16
-#define MOUSE_MOVES_ON "\033[?1003h"// Make the terminal report mouse movements
-#define MOUSE_MOVES_OFF "\033[?1003l"// Make the terminal report mouse movements
+#define MOUSE_MOVES_ON "\033[?1003h"
+#define MOUSE_MOVES_OFF "\033[?1003l"
+#define SGR_ON "\033[?1006h"
+#define SGR_OFF "\033[?1006l"
 #define STRING_ID_QUEUE_SIZE 8		// must be > 1
 #define COMMAND_QUEUE_SIZE 1280
 #define CMD_SIZE(x) sizeof(x)/sizeof(x[0])
@@ -141,6 +143,7 @@ typedef struct qwaq_resources_s {
 	PR_RESMAP (panel_t) panel_map;
 	cond_t      event_cond;
 	RING_BUFFER (qwaq_event_t, QUEUE_SIZE) event_queue;
+	qwaq_button button_state;
 	cond_t      command_cond;
 	RING_BUFFER (int, COMMAND_QUEUE_SIZE) command_queue;
 	cond_t      results_cond;
@@ -820,36 +823,70 @@ static void
 mouse_event (qwaq_resources_t *res, const MEVENT *mevent)
 {
 	int         mask = mevent->bstate;
+	qwaq_button buttons = res->button_state;
 	qwaq_event_t event = {};
 
+	if (mevent->bstate & ~M_MOVE) {
+		mvprintw (1, 0, "%08x", mevent->bstate);
+		doupdate();
+	}
 	event.mouse.x = mevent->x;
 	event.mouse.y = mevent->y;
-	event.mouse.buttons = mevent->bstate;
 	if (mask & M_MOVE) {
 		event.what = qe_mousemove;
-		mask &= ~M_MOVE;
 	}
 	if (mask & M_PRESS) {
 		event.what = qe_mousedown;
-		mask &= ~M_PRESS;
+		if (mask & BUTTON1_PRESSED) {
+			buttons |= qe_mouse1;
+		}
+		if (mask & BUTTON2_PRESSED) {
+			buttons |= qe_mouse2;
+		}
+		if (mask & BUTTON3_PRESSED) {
+			buttons |= qe_mouse3;
+		}
+#if 0	// wheel events don't report release
+		if (mask & BUTTON4_PRESSED) {
+			buttons |= qe_mouse4;
+		}
+		if (mask & BUTTON5_PRESSED) {
+			buttons |= qe_mouse5;
+		}
+#endif
 	}
 	if (mask & M_RELEASE) {
 		event.what = qe_mouseup;
-		mask &= ~M_RELEASE;
+		if (mask & BUTTON1_RELEASED) {
+			buttons &= ~qe_mouse1;
+		}
+		if (mask & BUTTON2_RELEASED) {
+			buttons &= ~qe_mouse2;
+		}
+		if (mask & BUTTON3_RELEASED) {
+			buttons &= ~qe_mouse3;
+		}
+#if 0	// wheel events don't report release
+		if (mask & BUTTON4_RELEASED) {
+			buttons &= ~qe_mouse4;
+		}
+		if (mask & BUTTON5_RELEASED) {
+			buttons &= ~qe_mouse5;
+		}
+#endif
 	}
+	event.mouse.buttons = buttons;
+	res->button_state = buttons;
 	if (mask & M_CLICK) {
 		event.what = qe_mouseclick;
-		mask &= ~M_CLICK;
 		event.mouse.click = 1;
 	}
 	if (mask & M_DCLICK) {
 		event.what = qe_mouseclick;
-		mask &= ~M_DCLICK;
 		event.mouse.click = 2;
 	}
 	if (mask & M_TCLICK) {
 		event.what = qe_mouseclick;
-		mask &= ~M_TCLICK;
 		event.mouse.click = 3;
 	}
 	add_event (res, &event);