From 09e923d965a86a944dad7d77b336721386f6bf63 Mon Sep 17 00:00:00 2001
From: Bent Bisballe Nyeng <deva@aasimon.org>
Date: Fri, 10 Feb 2017 18:39:27 +0100
Subject: Return a list of events form native window instead of one event at a
 time.

---
 plugingui/nativewindow_x11.cc | 67 ++++++-------------------------------------
 1 file changed, 9 insertions(+), 58 deletions(-)

(limited to 'plugingui/nativewindow_x11.cc')

diff --git a/plugingui/nativewindow_x11.cc b/plugingui/nativewindow_x11.cc
index f50e01d..3fffcd5 100644
--- a/plugingui/nativewindow_x11.cc
+++ b/plugingui/nativewindow_x11.cc
@@ -264,52 +264,21 @@ void NativeWindowX11::grabMouse(bool grab)
 	// Don't need to do anything on this platform...
 }
 
-bool NativeWindowX11::hasEvent()
+std::queue<std::shared_ptr<Event>> NativeWindowX11::getEvents()
 {
-	if(display == nullptr)
-	{
-		return false;
-	}
-
-	return XPending(display);
-}
-
-std::shared_ptr<Event> NativeWindowX11::getNextEvent()
-{
-	if(display == nullptr)
-	{
-		return nullptr;
-	}
-
-	if(!XPending(display))
-	{
-		return nullptr;
-	}
-
-	XEvent xEvent;
-	XNextEvent(display, &xEvent);
-	return translateXMessage(xEvent);
-}
+	std::queue<std::shared_ptr<Event>> events;
 
-std::shared_ptr<Event> NativeWindowX11::peekNextEvent()
-{
-	if(display == nullptr)
+	while(XPending(display))
 	{
-		return nullptr;
+		XEvent xEvent;
+		XNextEvent(display, &xEvent);
+		events.push(translateXMessage(xEvent));
 	}
 
-	if(!XPending(display))
-	{
-		return nullptr;
-	}
-
-	XEvent peekXEvent;
-	XPeekEvent(display, &peekXEvent);
-	return translateXMessage(peekXEvent, true);
+	return events;
 }
 
-std::shared_ptr<Event> NativeWindowX11::translateXMessage(XEvent& xevent,
-                                                          bool peek)
+std::shared_ptr<Event> NativeWindowX11::translateXMessage(XEvent& xevent)
 {
 	std::shared_ptr<Event> event = nullptr;
 
@@ -327,15 +296,6 @@ std::shared_ptr<Event> NativeWindowX11::translateXMessage(XEvent& xevent,
 
 	case Expose:
 		//DEBUG(x11, "Expose");
-		//if(!peek)
-		//{
-		//	XEvent peekXEvent;
-		//	while(XCheckWindowEvent(display, xwindow, Expose, &peekXEvent))
-		//	{
-		//		xevent.xexpose = peekXEvent.xexpose;
-		//	}
-		//}
-
 		if(xevent.xexpose.count == 0)
 		{
 			auto repaintEvent = std::make_shared<RepaintEvent>();
@@ -351,15 +311,6 @@ std::shared_ptr<Event> NativeWindowX11::translateXMessage(XEvent& xevent,
 	case ConfigureNotify:
 		//DEBUG(x11, "ConfigureNotify");
 		{
-			//if(!peek)
-			//{
-			//	XEvent peekXEvent;
-			//	while(XCheckWindowEvent(display, xwindow, ConfigureNotify, &peekXEvent))
-			//	{
-			//		xevent.xconfigure = peekXEvent.xconfigure;
-			//	}
-			//}
-
 			if((window.width() != (std::size_t)xevent.xconfigure.width) ||
 			   (window.height() != (std::size_t)xevent.xconfigure.height))
 			{
@@ -435,7 +386,7 @@ std::shared_ptr<Event> NativeWindowX11::translateXMessage(XEvent& xevent,
 					(xevent.type == ButtonPress) &&
 					((xevent.xbutton.time - last_click) < 200);
 
-				if(!peek && (xevent.type == ButtonPress))
+				if(xevent.type == ButtonPress)
 				{
 					last_click = xevent.xbutton.time;
 				}
-- 
cgit v1.2.3