simon-git: xtruss (main): Simon Tatham

[Commits to Tartarus hosted VCS]

TL;DR:
  0638eee Remove the pointless EXPECT_EVENT macro.
  ddbb8eb Window grabbing: tolerate MappingNotify events.

Repository:     https://git.tartarus.org/simon/xtruss.git
On the web:     https://git.tartarus.org/?p=simon/xtruss.git
Branch updated: main
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2024-11-10 14:23:00

commit 0638eee65275b6cfe8a247ce4c48cc81a0de5985
web diff https://git.tartarus.org/?p=simon/xtruss.git;a=commitdiff;h=0638eee65275b6cfe8a247ce4c48cc81a0de5985;hp=c25bf48c14c977d1c76f122b73c37f15df6ae6d5
Author: Simon Tatham <anakin at pobox.com>
Date:   Sun Nov 10 14:17:33 2024 +0000

    Remove the pointless EXPECT_EVENT macro.
    
    It was only used once, immediately after its definition, so there's no
    real need to have a macro at all, and improves legibility not to bother.
    
    No functional change. I've just expanded out the macro call, removed
    the macro definition, and folded in the crReadUpTo from just above it
    so that it lives in the appropriate branch of the if-else chain
    instead of in a separate if of its own. One line was rewrapped.

 xtruss-record.c | 46 ++++++++++++++++++++++------------------------
 1 file changed, 22 insertions(+), 24 deletions(-)

commit ddbb8eba514dfb839617745f70e9c4d31d597472
web diff https://git.tartarus.org/?p=simon/xtruss.git;a=commitdiff;h=ddbb8eba514dfb839617745f70e9c4d31d597472;hp=0638eee65275b6cfe8a247ce4c48cc81a0de5985
Author: Simon Tatham <anakin at pobox.com>
Date:   Sun Nov 10 14:18:43 2024 +0000

    Window grabbing: tolerate MappingNotify events.
    
    Apparently MappingNotify can be sent unsolicited, even to X clients
    who didn't ask for any such event. Deliberately ignore it during the
    'xtruss -p -' event loop that waits for the user to click a window.
    
    This patch reindents a lot of code by moving it into a while loop, but
    if you view it with whitespace off, there's only one small change in
    the code other than the reindentation.

 xtruss-record.c | 56 ++++++++++++++++++++++++++++++++------------------------
 1 file changed, 32 insertions(+), 24 deletions(-)