X-Git-Url: http://git.hungrycats.org/cgi-bin/gitweb.cgi?p=xscreensaver;a=blobdiff_plain;f=driver%2Ftimers.c;h=5f1bd418fc3b82ec53c6925d71a73181a6b971f4;hp=982b073291687780549080a7bfe161b4bfee158a;hb=a719ec12b8b2563112366a8ac3196816fd64d2c7;hpb=59ac4e9a0de290e4275a7bbb890ad16abd09d68f

diff --git a/driver/timers.c b/driver/timers.c
index 982b0732..5f1bd418 100644
--- a/driver/timers.c
+++ b/driver/timers.c
@@ -118,7 +118,16 @@ notice_events (saver_info *si, Window window, Bool top_p)
 
   /* Select for SubstructureNotify on all windows.
      Select for KeyPress on all windows that already have it selected.
-     Do we need to select for ButtonRelease?  I don't think so.
+
+     Note that we can't select for ButtonPress, because of X braindamage:
+     only one client at a time may select for ButtonPress on a given
+     window, though any number can select for KeyPress.  Someone explain
+     *that* to me.
+
+     So, if the user spends a while clicking the mouse without ever moving
+     the mouse or touching the keyboard, we won't know that they've been
+     active, and the screensaver will come on.  That sucks, but I don't
+     know how to get around it.
    */
   XSelectInput (si->dpy, window, SubstructureNotifyMask | events);