xf86-input-mouse: Changes to 'master'

[GitLab Mirror]

 .gitlab-ci.yml  |    4 ++--
 configure.ac    |    2 +-
 src/mouse.c     |   28 ++++++++++++++++++----------
 src/sun_mouse.c |   13 ++++++++-----
 4 files changed, 29 insertions(+), 18 deletions(-)

New commits:
commit 29456a2ad33897b0834aa097390cdfe5e92d1364
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 12:19:47 2022 -0700

    configure: check for timingsafe_memcmp
    
    Not needed by mouse driver, but quiets warnings from xorg server headers:
    
    In file included from /usr/include/xorg/misc.h:117:0,
                     from /usr/include/xorg/xf86str.h:37,
                     from /usr/include/xorg/xf86.h:44,
                     from mouse.c:57:
    /usr/include/xorg/os.h:595:1: warning: redundant redeclaration of ‘timingsafe_memcmp’ [-Wredundant-decls]
     timingsafe_memcmp(const void *b1, const void *b2, size_t len);
     ^~~~~~~~~~~~~~~~~
    In file included from mouse.c:52:0:
    /usr/include/string.h:235:12: note: previous declaration of ‘timingsafe_memcmp’ was here
     extern int timingsafe_memcmp(const void *s1, const void *s2, size_t n);
                ^~~~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit cbb97f4249ac6768e4a68423825ef0394104f5d8
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 12:14:57 2022 -0700

    autoGood: quiet -Wimplicit-fallthrough warning
    
    mouse.c: In function ‘autoGood’:
    mouse.c:3724:12: warning: this statement may fall through [-Wimplicit-fallthrough=]
             if (mPriv->goodCount < PROBE_UNCERTAINTY/2)
                ^
    mouse.c:3726:5: note: here
         default:
         ^~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit b888251a032dfff5f7219334d2e8f256251fa98e
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 12:07:37 2022 -0700

    InputDriverRec: Fix -Wmissing-field-initializers warning
    
    mouse.c:185:1: warning: missing initializer for field ‘default_options’ of ‘InputDriverRec {aka struct _InputDriverRec}’ [-Wmissing-field-initializers]
     };
     ^
    In file included from mouse.c:63:0:
    /usr/include/xorg/xf86Xinput.h:83:18: note: ‘default_options’ declared here
         const char **default_options;
                      ^~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 03fb2ae36be7314416b1c57b391039fb948468eb
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 12:02:51 2022 -0700

    SetupMouse: fix -Wsign-compare warning
    
    mouse.c: In function ‘SetupMouse’:
    mouse.c:2620:27: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
                 for (i = 0; i < sizeof(pMse->protoPara); i++)
                               ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit a8ff2170cb89145056d6f30a96219393316f1a96
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 11:58:47 2022 -0700

    checkForErraticMovements: Fix -Wempty-body warnings
    
    mouse.c: In function ‘checkForErraticMovements’:
    mouse.c:3759:52: warning: suggest braces around empty body in an ‘else’ statement [-Wempty-body]
                     AP_DBG(("accDx=%i\n",mPriv->accDx));
                                                        ^
    mouse.c:3772:52: warning: suggest braces around empty body in an ‘else’ statement [-Wempty-body]
                     AP_DBG(("accDy=%i\n",mPriv->accDy));
                                                        ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 72323bf07cc74fb03e63898b14136ae30a770580
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 11:53:20 2022 -0700

    sun_mouse: Fix -Wnull-dereference warning
    
    sun_mouse.c: In function ‘vuidReadInput’:
    sun_mouse.c:291:10: warning: potential null pointer dereference [-Wnull-dereference]
         pBuf = pVuidMse->buffer;
         ~~~~~^~~~~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit f62f35747ec65e7f8819fab2c928db1b1b4c56eb
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Oct 16 11:47:18 2022 -0700

    sun_mouse: Fix -Wsign-compare warnings
    
    sun_mouse.c: In function ‘vuidReadInput’:
    sun_mouse.c:299:22: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             } else if (n == -1) {
                          ^~
    sun_mouse.c: In function ‘vuidMouseProc’:
    sun_mouse.c:507:33: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             if (vuidMouseGeneration != serverGeneration) {
                                     ^~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 2d963a9f619420834274cedf407b754caecbccb3
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Jul 28 17:30:21 2022 -0700

    gitlab CI: stop requiring Signed-off-by in commits
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>