[patch] Upstream fix for similar problem

control-center-2.12.3-buttons.patch (text/plain), 2.54 KB, created by Daniel Gryniewicz (RETIRED) on 2006-07-06 18:00:18 UTC

diff --exclude-from=/home/dang/.diffrc -u -ruN control-center-2.12.3.orig/gnome-settings-daemon/gnome-settings-mouse.c control-center-2.12.3/gnome-settings-daemon/gnome-settings-mouse.c
--- control-center-2.12.3.orig/gnome-settings-daemon/gnome-settings-mouse.c	2005-11-02 06:49:32.000000000 -0500
+++ control-center-2.12.3/gnome-settings-daemon/gnome-settings-mouse.c	2006-07-06 20:57:23.000000000 -0400
@@ -33,24 +33,57 @@
                          gint      n_buttons,
                          gboolean  left_handed)
 {
-  const gint left_button = 0;
+  const gint left_button = 1;
   gint right_button;
+  gint i;
 
   /* if the button is higher than 2 (3rd button) then it's
    * probably one direction of a scroll wheel or something else
    * uninteresting
    */
-  right_button = MIN (n_buttons - 1, 2);
+  right_button = MIN (n_buttons, 3);
+
+  /* If we change things we need to make sure we only swap buttons.
+   * If we end up with multiple physical buttons assigned to the same
+   * logical button the server will complain. This code assumes physical
+   * button 0 is the physical left mouse button, and that the physical
+   * button other than 0 currently assigned left_button or right_button
+   * is the physical right mouse button.
+   */
+
+  /* check if the current mapping satisfies the above assumptions */
+  if (buttons[left_button - 1] != left_button && 
+      buttons[left_button - 1] != right_button)
+      /* The current mapping is weird. Swapping buttons is probably not a
+       * good idea.
+       */
+      return;
 
-  if (left_handed)
+  /* check if we are left_handed and currently not swapped */
+  if (left_handed && buttons[left_button - 1] == left_button)
     {
-      buttons[left_button] = right_button + 1;
-      buttons[right_button] = left_button + 1;
+      /* find the right button */
+      for (i = 0; i < n_buttons; i++)
+        {
+          if (buttons[i] == right_button)
+              break;
+      }
+      /* swap the buttons */
+      buttons[left_button - 1] = right_button;
+      buttons[i] = left_button;
     }
-  else
+  /* check if we are not left_handed but are swapped */
+  else if (!left_handed && buttons[left_button - 1] == right_button)
     {
-      buttons[left_button] = left_button + 1;
-      buttons[right_button] = right_button + 1;
+      /* find the right button */
+      for (i = 0; i < n_buttons; i++)
+        {
+          if (buttons[i] == left_button)
+              break;
+      }
+      /* swap the buttons */
+      buttons[i] = right_button;
+      buttons[left_button - 1] = left_button;
   }
 }