From e02fe34e17572a60f955ea5688a6a30ae9061b54 Mon Sep 17 00:00:00 2001
From: David Edmundson <kde@davidedmundson.co.uk>
Date: Fri, 19 Jun 2020 00:00:32 +0100
Subject: [PATCH] [wayland] Avoid debug recursion in XdgShellClients

QDebug& operator<<(QDebug& stream, TopLevel*)
calls TopLevel::debug(stream)

So our implementation has to provide something custom not call the base
class.

Debug here is loosely based on X11Client. It probably can be expanded on
in time, but at least it won't crash.
---
 xdgshellclient.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/xdgshellclient.cpp b/xdgshellclient.cpp
index df9bd92bf4..56a7760ee8 100644
--- a/xdgshellclient.cpp
+++ b/xdgshellclient.cpp
@@ -676,7 +676,7 @@ XdgToplevelInterface *XdgToplevelClient::shellSurface() const
 
 void XdgToplevelClient::debug(QDebug &stream) const
 {
-    stream << this;
+    stream << "XdgTopLevel: " << resourceClass() << caption();
 }
 
 NET::WindowType XdgToplevelClient::windowType(bool direct, int supported_types) const
@@ -1817,7 +1817,7 @@ XdgPopupClient::~XdgPopupClient()
 
 void XdgPopupClient::debug(QDebug &stream) const
 {
-    stream << this;
+    stream << "XdgPopup: transientFor:" << transientFor();
 }
 
 NET::WindowType XdgPopupClient::windowType(bool direct, int supported_types) const