Clipboard.c: rename function to better reflect meaning

Also improve Debug output.

1 files changed, 12 insertions, 17 deletions

diff --git a/nx-X11/programs/Xserver/hw/nxagent/Clipboard.c b/nx-X11/programs/Xserver/hw/nxagent/Clipboard.c
index fefb0286e..e53b454da 100644
--- a/nx-X11/programs/Xserver/hw/nxagent/Clipboard.c
+++ b/nx-X11/programs/Xserver/hw/nxagent/Clipboard.c
@@ -1699,10 +1699,10 @@ static void resetSelectionOwnerOnXServer(void)
     clearSelectionOwnerData(index);
 
     setClientSelectionStage(SelectionStageNone, index);
-  }
 
-  /* Hmm, this is already None when reaching this */
-  lastServers[index].requestor = None;
+    /* Hmm, this is already None when reaching this */
+    lastServers[index].requestor = None;
+  }
 }
 #endif
 
@@ -1848,7 +1848,8 @@ static void setSelectionOwnerOnXServer(Selection *pSelection)
     setClientSelectionStage(SelectionStageNone, index);
   }
 
-  lastServers[index].requestor = None;
+  /* FIXME: commented because index is invalid here! */
+  /* lastServers[index].requestor = None; */
 
 /*
 FIXME
@@ -2226,12 +2227,8 @@ XlibAtom translateLocalToRemoteTarget(Atom local)
  * selection. When there is an owner, it should be generated by the
  * owner of the selection by using XSendEvent()."
  */
-int nxagentSendNotify(xEvent *event)
+int nxagentSendNotificationToSelfViaXServer(xEvent *event)
 {
-  #ifdef DEBUG
-  fprintf(stderr, "%s: Got called.\n", __func__);
-  #endif
-
   if (!agentClipboardInitialized)
   {
     #ifdef DEBUG
@@ -2241,13 +2238,11 @@ int nxagentSendNotify(xEvent *event)
   }
 
   #ifdef DEBUG
-  fprintf(stderr, "%s: property is [%d][%s].\n", __func__,
-          event->u.selectionNotify.property,
-          NameForAtom(event->u.selectionNotify.property));
-  fprintf(stderr, "%s: selection is [%d][%s].\n", __func__,
-          event->u.selectionNotify.selection,
-          NameForAtom(event->u.selectionNotify.selection));
-  fprintf(stderr, "%s: requestor is [0x%x].\n", __func__, event->u.selectionNotify.requestor);
+  fprintf(stderr, "%s: Received SendNotify by client: property [%d][%s] target [%d][%s] selection [%d][%s] requestor [0x%x] time [%u].\n", __func__,
+          event->u.selectionNotify.property, NameForAtom(event->u.selectionNotify.property),
+          event->u.selectionNotify.target, NameForAtom(event->u.selectionNotify.target),
+          event->u.selectionNotify.selection, NameForAtom(event->u.selectionNotify.selection),
+          event->u.selectionNotify.requestor, event->u.selectionNotify.time);
   #endif
 
   int index = nxagentFindCurrentSelectionIndex(event->u.selectionNotify.selection);
@@ -2403,7 +2398,7 @@ Bool nxagentInitClipboard(WindowPtr pWin)
 
   /*
    * Server side properties to hold pasted data.
-   * see nxagentSendNotify for an explanation
+   * see nxagentSendNotificationToSelfViaXServer for an explanation
    */
   serverTransFromAgentProperty = nxagentAtoms[15]; /* NX_SELTRANS_FROM_AGENT */
   serverTransToAgentProperty = nxagentAtoms[5];  /* NX_CUT_BUFFER_SERVER */