twm: Changes to 'master'

Wed Feb 28 02:18:26 UTC 2024

 src/add_window.c |   47 ++++++++++++++----------
 src/cursor.c     |    6 ++-
 src/events.c     |   59 ++++++++++++++++++++-----------
 src/events.h     |    2 -
 src/iconmgr.c    |   17 ++++----
 src/icons.c      |   40 ++++++++++++---------
 src/menus.c      |  104 ++++++++++++++++++++++++++++++-------------------------
 src/menus.h      |    1 
 src/parse.c      |   17 +++++---
 src/screen.h     |    2 -
 src/session.c    |    8 ++--
 src/session.h    |    2 -
 src/twm.c        |   38 +++++++-------------
 src/twm.h        |    6 ---
 src/util.c       |   18 ++++++---
 15 files changed, 205 insertions(+), 162 deletions(-)

New commits:
commit 783b31821d87ea551457f4700f796a51f359cfd1
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Remove global variables JunkWidth, JunkHeight, JunkBW, JunkDepth, JunkMask
    
    Replace with a local variable following the naming scheme from
    Identify() in src/menus.c.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit c5a63751f8b4eab72794e56365ead929023f2b94
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Remove global variables JunkX, JunkY
    
    Replace with a local variable following the naming scheme from
    Identify() in src/menus.c.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 6ea386bd98a737a9f54e2ca4216adf45868fd6e4
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Remove global variables JunkRoot, JunkChild
    
    Replace with a local variable following the naming scheme from
    Identify() in src/menus.c.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 08b916e7ab8ea54ee6c38c485f11c9d6da898355
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in ExecuteFunction
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 7b24db53fc606a94067d90e8c64d8e7c8c75c12b
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in IconUp
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 24bfa418612288c4847bcfe088aba05ce402d1d2
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in do_string_keyword
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 9ad7b060e055f08ff39ee569f1cf40c7ac56e321
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in CreateIconManagers
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit b9f33162eaad31988e3595ddcd3efd97ee2da0aa
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in HandleButtonPress
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit f6fb5b536cd2a96ef1ac26599fd48ef89a628a77
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in SetHighlightPixmap
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 895e0e6630eea3cea07d4e4f47f0264ea5263c8e
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Rework dummy variable usage in AddWindow
    
    twm defines several "junk" variables to use with functions like
    XQueryPointer or XGetGeometry. In some instances, the returned
    values are actually used, which makes the code confusing and hard
    to reason about. Use dedicated variables in those cases.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 0182535025b0864d6e0cf165ab14756bfbe6d015
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Make XtSignalId si static
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 81cb173c5001f28ad178e9a9f59f521a13e66e92
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Make FirstScreen a local variable
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit cf9a35db8381f9b809465306aec3bcfe3bc1dc0d
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Make appContext a local variable
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit d812d0884c926be4088a4e636807509e05d77a5f
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Make InitFile a local variable
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit b8fe4a7270502dd9242bd5585b67f3afbdbbbadb
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Make MultiScreen a local variable
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 90058211800fd45ee7cc8408278d3b200bdfd2e5
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Split up InitMenus
    
    Preparation for the removal of FirstScreen: Separate code that is
    executed for all screens from code that is executed for only the
    first screen.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>

commit 9e42567818fe26ed97e05cccc0f9b74a2f523ce6
Author: Tim Wiederhake <twied at gmx.net>
Date:   Sat Jan 20 16:07:00 2024 +0100

    Remove now unused NoPrintscreens variable
    
    Support for xprint was removed in commit 0ce52bc9ed.
    
    Signed-off-by: Tim Wiederhake <twied at gmx.net>