xkbprint: Changes to 'master'

[GitLab Mirror]

 configure.ac |    4 +++
 psgeom.c     |   73 +++++++++++++++++++++++++++++++----------------------------
 xkbprint.c   |   36 ++++++++++++++---------------
 3 files changed, 61 insertions(+), 52 deletions(-)

New commits:
commit 3ce73201e3d808e5e4f6c20e22d6e6e86e3b81e8
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jul 28 16:53:26 2024 -0700

    Use C99 struct initializers
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>

commit 4868f37f09843432b261371875b2f433f2a1f7fa
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jul 28 16:38:24 2024 -0700

    Variable scope reductions
    
    Some recommended by cppcheck, others found by manual inspection
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>

commit 823cecf574645cd26ed43b18c9c68a06dc25f95a
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jul 28 16:20:59 2024 -0700

    FindKeysymsByName: avoid NULL pointer dereference
    
    Clears warning from cppcheck:
    
    psgeom.c:1645:15: warning: Either the condition 'state->args!=0L' is
     redundant or there is possible null pointer dereference: state->args.
     [nullPointerRedundantCheck]
        if ((state->args->wantDiffs) && (eG >= XkbKeyNumGroups(xkb, kc)))
                  ^
    psgeom.c:1635:21: note: Assuming that condition 'state->args!=0L' is not
     redundant
        if (state->args != NULL) {
                        ^
    psgeom.c:1645:15: note: Null pointer dereference
        if ((state->args->wantDiffs) && (eG >= XkbKeyNumGroups(xkb, kc)))
                  ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>

commit ed190ca257c808b5d76babf01a5dd57c7d749c9d
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jul 28 16:18:02 2024 -0700

    ClearFontStuff: avoid NULL pointer dereference
    
    Clears warning from cppcheck:
    
    psgeom.c:400:5: warning: Either the condition 'stuff' is redundant or there
     is possible null pointer dereference: stuff. [nullPointerRedundantCheck]
        bzero(stuff, sizeof(FontStuff));
        ^
    psgeom.c:398:9: note: Assuming that condition 'stuff' is not redundant
        if (stuff && stuff->foundry)
            ^
    psgeom.c:400:5: note: Null pointer dereference
        bzero(stuff, sizeof(FontStuff));
        ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>

commit 75a78342999f5c98ba5e0d154a1c506497daac5d
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jul 28 16:13:11 2024 -0700

    configure: Use AC_SYS_LARGEFILE to enable large file support
    
    While keymap files or generated output should never be more than 2gb in
    size, they may be stored on filesystems with inodes or timestamps outside
    of the 32-bit range.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>