xfd: Changes to 'master'

[GitLab Mirror]

 grid.c |   33 ++++++++++++++++++---------------
 xfd.c  |   17 +++++++++++------
 2 files changed, 29 insertions(+), 21 deletions(-)

New commits:
commit 09834f092c6fb5ac4945bfd1b7b5bb228e42e04d
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Nov 12 15:23:52 2022 -0800

    Tag unused arguments to callback functions as _X_UNUSED
    
    Gets rid of 33 -Wunused-parameter warnings from clang
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit d9ba30a6f56b2bb7e6f1b125a3d2eaa871c96eba
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Nov 12 13:05:24 2022 -0800

    Handle -Wextra-semi-stmt warnings from clang
    
    Use do {...} while (0) idiom for pre-processor macros that define
    code blocks so that trailing semi-colons don't appear to be extraneous
    
    grid.c:305:38: warning: empty expression statement has no effect;
      remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
                CI_GET_CHAR_INFO_1D (fs, ch, cs);
                                                ^
    grid.c:311:40: warning: empty expression statement has no effect;
      remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
                CI_GET_CHAR_INFO_2D (fs, r, c, cs);
                                                  ^
    grid.c:488:46: warning: empty expression statement has no effect;
      remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
        donestr (XftColor, xftColor, XtRXftColor);
                                                 ^
    grid.c:546:42: warning: empty expression statement has no effect;
      remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
        donestr (XftFont *, font, XtRXftFont);
                                             ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>