From: Po Lu <luangruo@yahoo.com>
Date: Thu, 28 Jul 2022 01:25:37 +0000 (+0800)
Subject: Remove excessive synchronization from x_sync_with_move
X-Git-Tag: emacs-29.0.90~1447^2~711
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=f720630195357440af179607aa74e8ed14a58e83;p=emacs.git

Remove excessive synchronization from x_sync_with_move

* src/xterm.c (x_sync_with_move): Remove redundant XSync.  The X
server always handles requests in order.
---

diff --git a/src/xterm.c b/src/xterm.c
index 2067e8bb741..7b4a65e87f0 100644
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -25055,11 +25055,9 @@ x_sync_with_move (struct frame *f, int left, int top, bool fuzzy)
       current_left = 0;
       current_top = 0;
 
-      /* In theory, this call to XSync only needs to happen once, but in
-         practice, it doesn't seem to work, hence the need for the surrounding
-         loop.  */
-
-      XSync (FRAME_X_DISPLAY (f), False);
+      /* There is no need to call XSync (even when no window manager
+	 is present) because x_real_positions already does that
+	 implicitly.  */
       x_real_positions (f, &current_left, &current_top);
 
       if (fuzzy)