nxagent: remove additional keyboard handling from Reconnect.c

nx-X11/programs/Xserver/hw/nxagent/Keyboard.c

@@ -649,7 +649,6 @@ XkbError:
                   __func__, nxagentRemoteRules, nxagentRemoteModel, nxagentRemoteLayout, nxagentRemoteVariant, nxagentRemoteOptions);
           #endif
 
-          /* Only setup keycode conversion if we are NOT in clone mode */
           if (nxagentKeyboard && (strcmp(nxagentKeyboard, "clone") == 0))
           {
             SAFE_free(rules); rules = strdup(nxagentRemoteRules);
@@ -657,6 +656,14 @@ XkbError:
             SAFE_free(layout); layout = strdup(nxagentRemoteLayout);
             SAFE_free(variant); variant = strdup(nxagentRemoteVariant);
             SAFE_free(options); options = strdup(nxagentRemoteOptions);
+
+            /* Only setup keycode conversion if we are NOT in clone mode */
+            #ifdef DEBUG
+            fprintf(stderr, "%s: nxagentKeyboard is [%s] - disabling keycode conversion.\n", __func__,
+                        nxagentKeyboard);
+            #endif
+            nxagentChangeOption(KeycodeConversion, KeycodeConversionOff);
+
 	    /*
              * when cloning we do not want X2Go to set the keyboard
              * via a keyboard file generated by nxagent. The defined
@@ -670,7 +677,6 @@ XkbError:
           }
           else
           {
-            nxagentKeycodeConversionSetup();
             /*
              * Keyboard has always been tricky with nxagent. For that
              * reason X2Go offers "auto" keyboard configuration. You can
@@ -688,6 +694,7 @@ XkbError:
               nxagentWriteKeyboardFile(nxagentRemoteRules, nxagentRemoteModel, nxagentRemoteLayout, nxagentRemoteVariant, nxagentRemoteOptions);
             }
           }
+          nxagentKeycodeConversionSetup();
         }
         #ifdef DEBUG
         else

nx-X11/programs/Xserver/hw/nxagent/Reconnect.c

@@ -599,18 +599,20 @@ Bool nxagentReconnectSession(void)
     nxagentOldKeyboard = NULL;
   }
 
-  if (nxagentOption(ResetKeyboardAtResume) == 1 &&
-         (nxagentKeyboard  == NULL || nxagentOldKeyboard == NULL ||
+  /* Reset the keyboard only if we detect any changes. */
+  if (nxagentOption(ResetKeyboardAtResume) == 1)
+  {
+    if (nxagentKeyboard == NULL || nxagentOldKeyboard == NULL ||
             strcmp(nxagentKeyboard, nxagentOldKeyboard) != 0 ||
                 strcmp(nxagentKeyboard, "query") == 0 ||
-                     strcmp(nxagentKeyboard, "clone") == 0))
+                    strcmp(nxagentKeyboard, "clone") == 0)
     {
       if (nxagentResetKeyboard() == 0)
       {
         #ifdef WARNING
         if (nxagentVerbose == 1)
         {
-        fprintf(stderr, "nxagentReconnectSession: Failed to reset keyboard device.\n");
+          fprintf(stderr, "%s: Failed to reset keyboard device.\n", __func__);
         }
         #endif
 
@@ -621,7 +623,10 @@ Bool nxagentReconnectSession(void)
     }
     else
     {
-    nxagentKeycodeConversionSetup();
+      #ifdef DEBUG
+      fprintf(stderr, "%s: keyboard unchanged - skipping keyboard reset.\n", __func__);
+      #endif
+    }
   }
 
   nxagentXkbState.Initialized = 0;