libXtst: Changes to 'master'

[GitLab Mirror]

 src/XRecord.c |   33 +++++++++++++++------------------
 src/XTest.c   |   14 +++++++++-----
 2 files changed, 24 insertions(+), 23 deletions(-)

New commits:
commit 14a44d0e3e6c3d9e757e2fdd143587efe532f1e9
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Aug 20 10:39:35 2022 -0700

    Remove obsolete casts from Xmalloc() and Xcalloc() calls
    
    Not needed in C89 and later
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit afc015217221acd8667a96111b147f3a742a2c4e
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 18 19:12:42 2022 -0700

    Variable scope reductions as suggested by cppcheck
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit ff3456b04e1b42fd77e0db061bf7a563108b5868
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 18 19:02:35 2022 -0700

    Resolve -Wsign-compare warnings
    
    XRecord.c: In function ‘XRecordFreeState’:
    XRecord.c:515:15: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
         for(i=0; i<state->nclients; i++) {
                   ^
    XRecord.c: In function ‘parse_reply_call_callback’:
    XRecord.c:752:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 4 > rep->length << 2)
                             ^
    XRecord.c:759:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (current_index + 1 > rep->length << 2)
                                ^
    XRecord.c:763:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 8 > rep->length << 2)
                             ^
    XRecord.c:777:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 4 > rep->length << 2)
                             ^
    XRecord.c:785:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 4 > rep->length << 2)
                             ^
    XRecord.c:792:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (current_index + 4 > rep->length<<2)
                                ^
    XRecord.c:797:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 8 > rep->length << 2)
                             ^
    XRecord.c:810:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (current_index + 8 > rep->length << 2)
                                ^
    XRecord.c:818:25: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
       if (current_index + 4 > rep->length << 2)
                             ^
    XRecord.c:824:31: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          } else if (current_index < rep->length << 2)
                                   ^
    XRecord.c:830:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (current_index < rep->length << 2)
                            ^
    XRecord.c:859:27: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
         } while (current_index<rep->length<<2);
                               ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit f8a2329d8a24c0901d945986232267c02f080fc4
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 18 18:52:40 2022 -0700

    send_axes: Mark switch statement fallthrough as intentional
    
    Quiets gcc warnings:
    XTest.c: In function ‘send_axes’:
    XTest.c:274:19: warning: this statement may fall through [-Wimplicit-fallthrough=]
          ev.valuator5 = *(axes+5);
          ~~~~~~~~~~~~~^~~~~~~~~~~
    XTest.c:275:2: note: here
      case 5:
      ^~~~
    XTest.c:276:19: warning: this statement may fall through [-Wimplicit-fallthrough=]
          ev.valuator4 = *(axes+4);
          ~~~~~~~~~~~~~^~~~~~~~~~~
    XTest.c:277:2: note: here
      case 4:
      ^~~~
    XTest.c:278:19: warning: this statement may fall through [-Wimplicit-fallthrough=]
          ev.valuator3 = *(axes+3);
          ~~~~~~~~~~~~~^~~~~~~~~~~
    XTest.c:279:2: note: here
      case 3:
      ^~~~
    XTest.c:280:19: warning: this statement may fall through [-Wimplicit-fallthrough=]
          ev.valuator2 = *(axes+2);
          ~~~~~~~~~~~~~^~~~~~~~~~~
    XTest.c:281:2: note: here
      case 2:
      ^~~~
    XTest.c:282:19: warning: this statement may fall through [-Wimplicit-fallthrough=]
          ev.valuator1 = *(axes+1);
          ~~~~~~~~~~~~~^~~~~~~~~~~
    XTest.c:283:2: note: here
      case 1:
      ^~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>