From 98d03fb09846c62273973c2c80dfdef86c92d76e Mon Sep 17 00:00:00 2001 From: Nick Bolton Date: Tue, 5 Aug 2025 13:45:25 +0100 Subject: [PATCH] chore: Improve logging in Server::handleClipboardGrabbed --- src/lib/server/Server.cpp | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/lib/server/Server.cpp b/src/lib/server/Server.cpp index 031b0573d..bc2b87c71 100644 --- a/src/lib/server/Server.cpp +++ b/src/lib/server/Server.cpp @@ -1172,14 +1172,14 @@ void Server::handleClipboardGrabbed(const Event &event, BaseClientProxy *grabber // screen to grab. ClipboardInfo &clipboard = m_clipboards[info->m_id]; if (grabber != m_primaryClient && info->m_sequenceNumber < clipboard.m_clipboardSeqNum) { - LOG((CLOG_INFO "ignored screen \"%s\" grab of clipboard %d", getName(grabber).c_str(), info->m_id)); + LOG_INFO("ignored screen \"%s\" grab of clipboard %d", getName(grabber).c_str(), info->m_id); return; } // mark screen as owning clipboard - LOG( - (CLOG_INFO "screen \"%s\" grabbed clipboard %d from \"%s\"", getName(grabber).c_str(), info->m_id, - clipboard.m_clipboardOwner.c_str()) + LOG_INFO( + "screen \"%s\" grabbed clipboard %d from \"%s\"", getName(grabber).c_str(), info->m_id, + clipboard.m_clipboardOwner.c_str() ); clipboard.m_clipboardOwner = getName(grabber); clipboard.m_clipboardSeqNum = info->m_sequenceNumber; @@ -1203,8 +1203,7 @@ void Server::handleClipboardGrabbed(const Event &event, BaseClientProxy *grabber } if (grabber == m_primaryClient && m_active != m_primaryClient) { - LOG((CLOG_INFO "clipboard grabbed, but we are already changed active " - "screen. Resend clipboard data")); + LOG_INFO("clipboard grabbed while active screen was changed, resending clipboard data"); for (ClipboardID id = 0; id < kClipboardEnd; ++id) { onClipboardChanged(m_primaryClient, id, m_clipboards[id].m_clipboardSeqNum); }