editres: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Mon May 11 14:50:15 PDT 2015


 Makefile.am          |    6 -
 actions.c            |   72 ++++++++++-----------
 app-defaults/Editres |    6 -
 comm.c               |  172 +++++++++++++++++++++++++--------------------------
 configure.ac         |    6 -
 editres.c            |   30 ++++----
 editresP.h           |   31 ++++-----
 geometry.c           |  120 ++++++++++++++++++-----------------
 handler.c            |  111 ++++++++++++++++----------------
 man/editres.man      |    4 -
 setvalues.c          |   20 ++---
 svpopup.c            |   64 +++++++++---------
 utils.c              |  116 +++++++++++++++++-----------------
 widgets.c            |  172 +++++++++++++++++++++++++--------------------------
 wtree.c              |  116 +++++++++++++++++-----------------
 15 files changed, 523 insertions(+), 523 deletions(-)

New commits:
commit a415214bf282dcc8385f2101e6f8c018e188fa41
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 9 13:13:32 2015 -0700

    Fix -Wmissing-noreturn warnings
    
    editres.c:140:1: warning: function 'Syntax' could be declared with
     attribute 'noreturn' [-Wmissing-noreturn]
    handler.c:65:1: warning: function 'Quit' could be declared with
     attribute 'noreturn' [-Wmissing-noreturn]
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit a6733f76dad52644bd45a8674bf0ea97c6e424ac
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 9 12:47:23 2015 -0700

    Strip trailing whitespace
    
    'git diff -w' shows no changes for this commit
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit c33a44612947eb91579b24f98848812018dd8c16
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 9 11:38:18 2015 -0700

    Fix -Wsign-compare warnings
    
    actions.c:291:21: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for ( i = 0 ; i < num_table; i++ )
                      ~ ^ ~~~~~~~~~
    actions.c:299:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < num_table; ) {
                    ~ ^ ~~~~~~~~~
    actions.c:302:8: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
            if (i == (num_table - 1))
                ~ ^   ~~~~~~~~~~~~~
    actions.c:304:13: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
            else if (i < num_table)
                     ~ ^ ~~~~~~~~~
    
    geometry.c:150:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_nodes; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~
    geometry.c:380:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_flash_widgets; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    geometry.c:401:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_flash_widgets; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    geometry.c:423:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_flash_widgets; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    geometry.c:428:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_flash_widgets; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    handler.c:475:13: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        if (len > malloc_size) {
            ~~~ ^ ~~~~~~~~~~~
    handler.c:476:58: warning: implicit conversion changes signedness: 'int' to 'unsigned long' [-Wsign-conversion]
            malloc_string = XtRealloc(malloc_string, sizeof(char) * len);
                                                                  ~ ^~~
    handler.c:477:16: warning: implicit conversion changes signedness: 'int' to 'Cardinal' (aka 'unsigned int') [-Wsign-conversion]
            malloc_size = len;
                        ~ ^~~
    
    utils.c:167:37: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (node = top_node, i = 1 ; i < number; i++) {
                                      ~ ^ ~~~~~~
    utils.c:170:16: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
            for (j = 0; j < node->num_children; j++) {
                        ~ ^ ~~~~~~~~~~~~~~~~~~
    utils.c:215:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < node->num_children; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~
    utils.c:787:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < top_node->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~
    
    wtree.c:129:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < top->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~
    wtree.c:194:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < info->num_nodes; i++) {
                    ~ ^ ~~~~~~~~~~~~~~~
    wtree.c:216:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < info->num_nodes; i++)
                    ~ ^ ~~~~~~~~~~~~~~~
    wtree.c:270:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < top->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~
    wtree.c:339:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < num_active_nodes; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~
    wtree.c:386:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < node->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~
    wtree.c:446:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < node->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~
    wtree.c:478:16: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
            for (i = 0; i < node->num_children; i++) {
                        ~ ^ ~~~~~~~~~~~~~~~~~~
    wtree.c:607:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < tree_info->num_nodes; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~
    wtree.c:686:19: warning: comparison of integers of different signs: 'int' and 'Cardinal' (aka 'unsigned int') [-Wsign-compare]
        for (i = 0; i < node->num_children; i++)
                    ~ ^ ~~~~~~~~~~~~~~~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit af90a105ec97c6d856d2e8a80e8822c3e192d1e9
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 9 11:19:33 2015 -0700

    Replace remaining sprintf calls with snprintf
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 9ffefd5b3d38e9ed2b5ed59bc50e8a7da761fb77
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat May 9 11:17:38 2015 -0700

    Replace XtMalloc+sprintf pair with XtAsprintf call
    
    Raises required version of libXt to >= 1.0.99.1
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list