libICE: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Thu Aug 8 23:54:27 PDT 2013


 include/X11/ICE/ICEmsg.h |   12 ++++++------
 src/ICElibint.h          |    4 ++--
 src/accept.c             |   14 ++++++--------
 src/authutil.c           |   20 +++++++-------------
 src/connect.c            |   15 ++++++---------
 src/getauth.c            |    4 ++--
 src/iceauth.c            |    2 +-
 src/listen.c             |   25 +++++++++++--------------
 src/listenwk.c           |   19 ++++++++-----------
 src/misc.c               |   13 ++++++-------
 src/ping.c               |    2 +-
 src/process.c            |   35 +++++++++++++++++------------------
 src/protosetup.c         |    9 ++++-----
 src/register.c           |   27 ++++++++++-----------------
 src/replywait.c          |    5 ++---
 src/setauth.c            |    2 +-
 src/shutdown.c           |   12 ++++++------
 src/watch.c              |   13 ++++++-------
 18 files changed, 102 insertions(+), 131 deletions(-)

New commits:
commit 2312ee00402088307e69589c3d12529b5232df66
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 23:01:30 2013 -0700

    Make STORE_STRING cast strlen result to CARD16 when storing in CARD16
    
    Clears a number of clang warnings of the form:
    
    connect.c:328:6: warning: implicit conversion loses integer precision:
       'size_t' (aka 'unsigned long') to 'CARD16' (aka 'unsigned short')
       [-Wconversion]
                STORE_STRING (pData, _IceAuthNames[i]);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ./ICElibint.h:173:19: note: expanded from macro 'STORE_STRING'
        CARD16 _len = strlen (_string); \
               ~~~~   ^~~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 6d6aa84dc6acb2daa3ef7e20942c38a1416bf543
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 22:58:46 2013 -0700

    Cast assignments in IceErrorHeader() macro
    
    Clears many clang warnings about implicit conversions losing
    integer precision, such as from storing ints into CARD8 or CARD16.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 9450bb648ef98efd6f08ea7d14ab8b9ea6e7bb54
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 22:47:41 2013 -0700

    Make write_string call write_counted_string instead of copying it
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 84153eee4db318cd897c464e70cb9f8bc8e469f6
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 22:37:09 2013 -0700

    Mark input arguments to write_string functions as const
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 9a74512ffdc1628f1b87d2191439915c63b9104f
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 22:31:41 2013 -0700

    Stop casting return values from malloc
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 4033226105fa861ab5f0276850afc24c0fa45406
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Aug 8 22:17:27 2013 -0700

    Get rid of casts to (char *) in calls to free()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list