vcl/inc/displayconnectiondispatch.hxx           |    1 -
 vcl/source/helper/displayconnectiondispatch.cxx |    6 ------
 vcl/unx/generic/dtrans/X11_selection.cxx        |    7 +++----
 3 files changed, 3 insertions(+), 11 deletions(-)

New commits:
commit 2adcb38ab7c984098706125978ea878951f5ab87
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Fri Jul 4 16:46:37 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Sat Jul 5 07:45:48 2025 +0200

    vcl: Move connection identifer logic out of DisplayConnectionDispatch
    
    No need to have this as a member in
    DisplayConnectionDispatch, get it from the SalInstance
    directly when needed.
    
    Change-Id: I9223f3b59d0c811632d4cf66b1a1dbae73a032ca
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/187406
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/inc/displayconnectiondispatch.hxx 
b/vcl/inc/displayconnectiondispatch.hxx
index f8d41393b03b..0e7d4540e256 100644
--- a/vcl/inc/displayconnectiondispatch.hxx
+++ b/vcl/inc/displayconnectiondispatch.hxx
@@ -52,7 +52,6 @@ public:
 
     void addEventHandler(const rtl::Reference<DisplayEventHandler>& handler);
     void removeEventHandler(const rtl::Reference<DisplayEventHandler>& 
handler);
-    OUString getIdentifier();
 };
 
 }
diff --git a/vcl/source/helper/displayconnectiondispatch.cxx 
b/vcl/source/helper/displayconnectiondispatch.cxx
index 54e0358b7146..be050547a726 100644
--- a/vcl/source/helper/displayconnectiondispatch.cxx
+++ b/vcl/source/helper/displayconnectiondispatch.cxx
@@ -30,7 +30,6 @@ using namespace com::sun::star::awt;
 
 DisplayConnectionDispatch::DisplayConnectionDispatch()
 {
-    m_ConnectionIdentifier = 
ImplGetSVData()->mpDefInst->GetConnectionIdentifier();
 }
 
 DisplayConnectionDispatch::~DisplayConnectionDispatch()
@@ -76,11 +75,6 @@ void DisplayConnectionDispatch::removeEventHandler(
     std::erase(m_aHandlers, handler);
 }
 
-OUString DisplayConnectionDispatch::getIdentifier()
-{
-    return m_ConnectionIdentifier;
-}
-
 bool DisplayConnectionDispatch::dispatchEvent(const void* pEvent)
 {
     SolarMutexReleaser aRel;
diff --git a/vcl/unx/generic/dtrans/X11_selection.cxx 
b/vcl/unx/generic/dtrans/X11_selection.cxx
index edfef0920e9d..b9cbb39c392a 100644
--- a/vcl/unx/generic/dtrans/X11_selection.cxx
+++ b/vcl/unx/generic/dtrans/X11_selection.cxx
@@ -23,6 +23,7 @@
 #include <cstdlib>
 #include <thread>
 
+#include <salinst.hxx>
 #include <unx/saldisp.hxx>
 
 #include <unistd.h>
@@ -342,9 +343,7 @@ void SelectionManager::initialize()
     if( m_pDisplay )
         return;
 
-    OUString aUDisplay;
-    if( m_xDisplayConnection.is() )
-        aUDisplay = m_xDisplayConnection->getIdentifier();
+    const OUString aUDisplay = 
ImplGetSVData()->mpDefInst->GetConnectionIdentifier();
 
     OString aDisplayName( OUStringToOString( aUDisplay, 
RTL_TEXTENCODING_ISO_8859_1 ) );
 
commit 7e76b2d8feee2f230c0a7d186789d6da002f9d9d
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Fri Jul 4 16:36:10 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Sat Jul 5 07:45:39 2025 +0200

    gen: Update comment
    
    This is no longer an XEventHandler after
    
        Change-Id: Iaf08f4aee3fea99d9d5d5c907d107452812d187d
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Fri Jul 4 14:51:17 2025 +0200
    
            vcl: No longer use css::awt::XEventHandler
    
    Change-Id: Ibce42e9040fd716c7b85ff5ee8ae8a6905750165
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/187405
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/unx/generic/dtrans/X11_selection.cxx 
b/vcl/unx/generic/dtrans/X11_selection.cxx
index 5a5f0a38fa42..edfef0920e9d 100644
--- a/vcl/unx/generic/dtrans/X11_selection.cxx
+++ b/vcl/unx/generic/dtrans/X11_selection.cxx
@@ -325,7 +325,7 @@ void SelectionManager::initialize()
     {
         /*
          *  We will get the XEvents of the vcl event loop from the display 
connection by
-         *  registering us as XEventHandler on it.
+         *  registering ourselves as an event handler for it.
          *
          *  implementor's note:
          *  FIXME:

Reply via email to