From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: martin rudalics Newsgroups: gmane.emacs.bugs Subject: bug#49997: 27.2; idle-time reset when switching desktop-page Date: Wed, 18 Aug 2021 10:02:33 +0200 Message-ID: <0e03c073-2678-5f47-1a92-6a585c8dfe19@gmx.at> References: <87eeb0mkxa.fsf@a16n.net> <83tujwdwe0.fsf@gnu.org> <87sfzakczt.fsf@gnus.org> <83eeauaihs.fsf@gnu.org> <87a6likc5a.fsf@gnus.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------BB6E566BA5FFEA9FE4CAB48D" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="21387"; mail-complaints-to="usenet@ciao.gmane.io" Cc: pm@a16n.net, 49997@debbugs.gnu.org To: Lars Ingebrigtsen , Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Wed Aug 18 10:03:13 2021 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mGGXY-0005QI-PJ for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 18 Aug 2021 10:03:12 +0200 Original-Received: from localhost ([::1]:37710 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mGGXX-0006HT-KF for geb-bug-gnu-emacs@m.gmane-mx.org; Wed, 18 Aug 2021 04:03:11 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:37676) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mGGXP-0006Er-GF for bug-gnu-emacs@gnu.org; Wed, 18 Aug 2021 04:03:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:43213) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1mGGXP-0004Y7-9u for bug-gnu-emacs@gnu.org; Wed, 18 Aug 2021 04:03:03 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1mGGXP-0004GZ-5n for bug-gnu-emacs@gnu.org; Wed, 18 Aug 2021 04:03:03 -0400 X-Loop: help-debbugs@gnu.org Resent-From: martin rudalics Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 18 Aug 2021 08:03:03 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 49997 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: confirmed Original-Received: via spool by 49997-submit@debbugs.gnu.org id=B49997.162927376316350 (code B ref 49997); Wed, 18 Aug 2021 08:03:03 +0000 Original-Received: (at 49997) by debbugs.gnu.org; 18 Aug 2021 08:02:43 +0000 Original-Received: from localhost ([127.0.0.1]:54757 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mGGX4-0004Fe-Tx for submit@debbugs.gnu.org; Wed, 18 Aug 2021 04:02:43 -0400 Original-Received: from mout.gmx.net ([212.227.15.19]:42945) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mGGX3-0004FR-25 for 49997@debbugs.gnu.org; Wed, 18 Aug 2021 04:02:41 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1629273754; bh=uurHfM5zBnPMTZ/F5dslsg9zRoX7w5v3rpGn5PWvNZg=; h=X-UI-Sender-Class:Subject:To:Cc:References:From:Date:In-Reply-To; b=ZD9B2rBW6J5TPJRz9aFcE/NIvbeiF3zy2jGKJk5YNmFjYAPjmiJjuqaCIjDQ1z4tY QTcWMDVRp3emh09NROOTyHvfU0wQU1xmrBERoywT3Aa1QCU4mvEBUk3IEh6ymQd52+ D9jGGvnxLi8P/DrbP9mRFMItCf+6eK2fbBtmi+ZI= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Original-Received: from [192.168.1.101] ([212.95.5.120]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1McH5a-1mm0qU2EQO-00chN5; Wed, 18 Aug 2021 10:02:34 +0200 In-Reply-To: <87a6likc5a.fsf@gnus.org> Content-Language: en-US X-Provags-ID: V03:K1:O3dNojgdCZbYoSnhcQivUhqzxWZixFjaLLjFZRZ6t9u4WTdXHB/ Cuj0OJLgPo2fYjec3/4h8nKDqaxGUp3ylmjVCL6hvfNcY9j/4xVid0L0gGmLs+QTH4QcfbW DXZInbFxFKs+hTrkAt20nXeOZgqByTWIHu7cj9hsrbU/x2NZe93N043LpbM4Zhbk4yl0MO4 YPc/fx+62PmZdaWc1Bg2g== X-UI-Out-Filterresults: notjunk:1;V03:K0:85btum/ss6k=:0qtFnw7TScnr5v/lc3AUaU s25u2r09w7R0FDUuF0V1IX26/aFraat/RKqDfefDctIAlqbmq202cZsdM7hCdS+X9INjwnaLO dA8wQK+VlbdojiScWpXdoU727ZyL+ohg79/dLojjWDykQD3Tywyjfrn8u6jcdL/cQk1kI/C9z j8u/+zBpKexeBtt2IqNKAtLRb1eGNS8eoZURJoAmSUUvPYw6ZvsZveUKMeA5SQiXpf0VaaP2y SopPrOTdfgh0oP80Ba1fzFBhFhvsl0DWMgSVir9PkZmlwUGbfD166CAgrotHrNSaJE0h/wEeM q0P7Bbuzt4RDnv1CDCdjBLDidoAdKESPTFQtEUPJQmbf0opEOUmlUuojXAeRQVCOnQuuuhezT aOwT0rnkr3G3o4eGpMUKnEx9e8aBJCjNciQHCIoASstxaJu+ORYCOiG3vY4Bm93wSsRUAybir 38sr1wMpxC5/DsTL8o1CYywCKf8PFtZIOvgUM3Z6NT4tTI5OMZovTu+JueNkvY9b8l3wdNnEf 8p4UhpfKV/qYK/WXd5fFLTlQeduBnKa2CyfWWk7qp1q7P6uJnVovEULtpAo8n9x5rK3WXaLhS 7s8KHo2qkr95qk7Qf7QkmuFAMTCY3y9OY9UOmAs7NUQLloy6Zm2lqN62HtF6+E9FbCJgV/hk5 kRTVvOTNtj7NPI6das4Y1WroUpRQDrw3AoGRwFzMwnZVWDVWQ/UO3PEzpU0ke/MB5JIJsRD0T RyHD/2aJHz23pX8Q0z8exOWYzu/8EKFhndgeUifs+eoYgAH2rZifYizhIgqJb3O+jNX+gTVm X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:212116 Archived-At: This is a multi-part message in MIME format. --------------BB6E566BA5FFEA9FE4CAB48D Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit >> The code you show is in keyboard.c, where we interpret the events >> we've received. To see whether we can distinguish between these >> events and "real" move-frame events, we need to look in xterm.c, where >> the events come in from the window-system. Maybe they are different >> in their raw form? > > Ah, I see. Right, this is in handle_one_xevent, where we apparently > synthesise the MOVE_FRAME_EVENT: > > if (!FRAME_TOOLTIP_P (f) > && (old_left != f->left_pos || old_top != f->top_pos)) > { > inev.ie.kind = MOVE_FRAME_EVENT; > XSETFRAME (inev.ie.frame_or_window, f); > } > } > > So it's purely based on whether the window manager told is that the > position changed -- which I guess it sort of does? When I move to a > different virtual desktop, it shows me all the iconified frames, and > that's probably where this comes from? > > This is in: > > case ConfigureNotify: > > that case in itself is almost 200 lines long... > > I've added Martin to the CCs; perhaps he has some insights here. I can offer the attached trivial patch. Peter, can you try it? martin --------------BB6E566BA5FFEA9FE4CAB48D Content-Type: text/x-patch; name="move-frame.diff" Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="move-frame.diff" diff --git a/src/keyboard.c b/src/keyboard.c index 34c64b9186..e643d3267b 100644 =2D-- a/src/keyboard.c +++ b/src/keyboard.c @@ -2929,6 +2929,7 @@ read_char (int commandflag, Lisp_Object map, if (CONSP (c) && (EQ (XCAR (c), Qselect_window) + || EQ (XCAR (c), Qmove_frame) || EQ (XCAR (c), Qfocus_out) #ifdef HAVE_DBUS || EQ (XCAR (c), Qdbus_event) --------------BB6E566BA5FFEA9FE4CAB48D--