mkfontscale: Changes to 'master'

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Nov 17 04:25:39 UTC 2018


 Makefile.am   |    2 ++
 README        |   26 --------------------------
 README.md     |   18 ++++++++++++++++++
 configure.ac  |    3 +++
 list.c        |   10 ++++++++++
 mkfontscale.c |   27 ++++++++++++++-------------
 6 files changed, 47 insertions(+), 39 deletions(-)

New commits:
commit 17dec8c3c2805707767a2e416358db860b2b2427
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Nov 16 20:25:32 2018 -0800

    Update README for gitlab migration
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit eac885fa5ebff24e0839144863872443a010d7c2
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Oct 1 21:24:45 2018 -0700

    Stop shadowing 'found' in checkExtraEncoding()
    
    mkfontscale.c: In function ‘checkExtraEncoding’:
    mkfontscale.c:1220:17: warning: declaration of ‘found’ shadows a parameter [-Wshadow]
                 int found = 0;
                     ^~~~~
    mkfontscale.c:1214:59: note: shadowed declaration is here
     checkExtraEncoding(FT_Face face, char *encoding_name, int found)
                                                               ^~~~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 325621bd083236567bd861d41ff161313fc96e96
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Oct 1 21:19:58 2018 -0700

    Rename variable 'n' to avoid shadowing another 'n' in doDirectory
    
    mkfontscale.c: In function ‘doDirectory’:
    mkfontscale.c:928:17: warning: declaration of ‘n’ shadows a previous local [-Wshadow]
                 int n = strlen(xlfd_name);
                     ^
    mkfontscale.c:788:12: note: shadowed declaration is here
         int i, n, dirn, diri, found, rc;
                ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 0faf882de6de6c1ec26aace9a9939914b40e7b89
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Oct 1 21:10:59 2018 -0700

    Use vasprintf(), if available, to implement dsprintf()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 3b747cc5d2981c44ff517f64c355e76d70194f24
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Oct 1 20:52:18 2018 -0700

    Replace dsprintf("%s", s) with strdup(s)
    
    dsprintf is an allocating sprintf.  With a format string of "%s", it
    reduces to just being a more complicated version of strdup, so lets
    just use strdup instead to save time and reduce complexity.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 65fe4ef6f5674b0b52e797924990b090dbbd52af
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Mon Oct 1 20:48:21 2018 -0700

    Remove last use of strcat
    
    If getcwd returned a string that exactly filled the prefix buffer,
    it could write the null terminator byte out of bounds.
    
    Instead, since the prefix is copied to a newly allocated string and
    never used again, just let the dsprintf() call take care of it.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list