* term.c (term_mouse_position): Don't assume time_t wraparound.

diff --git a/src/ChangeLog b/src/ChangeLog
index a772106c5211bc53a0fe39430e1ee0c7e81bcff0..21c03ba82200b9e2b0c28ce4706dc57a7b11fcf4 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,7 @@
 2011-05-12  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * term.c (term_mouse_position): Don't assume time_t wraparound.
+
        Be more systematic about user-interface timestamps.
        Before, the code sometimes used 'Time', sometimes 'unsigned long',
        and sometimes 'EMACS_UINT', to represent these timestamps.  This
@@ -14,6 +16,7 @@
        * menu.c (Fx_popup_menu) [!HAVE_X_WINDOWS]: Likewise.
        * menu.h (xmenu_show): Likewise.
        * term.c (term_mouse_position): Likewise.
+
        * termhooks.h (struct input_event.timestamp): Likewise.
        (struct terminal.mouse_position_hook): Likewise.
        * xmenu.c (create_and_show_popup_menu, xmenu_show): Likewise.

diff --git a/src/term.c b/src/term.c
index 34320a1ad6dea0b13ef8b0e9e84b256754d0510a..5fe258caa2933e105b75b18e347518e2c587851a 100644 (file)
--- a/src/term.c
+++ b/src/term.c
@@ -2701,6 +2701,7 @@ term_mouse_position (FRAME_PTR *fp, int insist, Lisp_Object *bar_window,
                     Lisp_Object *y, Time *timeptr)
 {
   struct timeval now;
+  Time sec, usec;
 
   *fp = SELECTED_FRAME ();
   (*fp)->mouse_moved = 0;
@@ -2711,7 +2712,9 @@ term_mouse_position (FRAME_PTR *fp, int insist, Lisp_Object *bar_window,
   XSETINT (*x, last_mouse_x);
   XSETINT (*y, last_mouse_y);
   gettimeofday(&now, 0);
-  *timeptr = (now.tv_sec * 1000) + (now.tv_usec / 1000);
+  sec = now.tv_sec;
+  usec = now.tv_usec;
+  *timeptr = (sec * 1000) + (usec / 1000);
 }
 
 /* Prepare a mouse-event in *RESULT for placement in the input queue.