bdftopcf: Changes to 'master'

[GitLab Mirror]

 atom.c       |   32 +++++++-------------------
 bdfint.h     |    1 
 bdfread.c    |   52 ++++++++++++-------------------------------
 bdfutils.c   |   17 +++++---------
 bitmap.c     |   19 ++++++---------
 bitmaputil.c |   27 +++++++++-------------
 bufio.c      |   33 +++++----------------------
 bufio.h      |    6 ----
 fntfilio.h   |    1 
 fontink.c    |    9 +++----
 fontmisc.h   |    2 -
 pcf.h        |    1 
 pcfwrite.c   |   12 ++++-----
 private.c    |   63 ----------------------------------------------------
 stubs.h      |    1 
 utilbitmap.c |   71 ++++-------------------------------------------------------
 16 files changed, 71 insertions(+), 276 deletions(-)

New commits:
commit 7e29b1d29c6d7cbabe1bad536f7b8f644bffcd4f
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 16:05:28 2022 -0700

    Remove unnecessary cast of strlen result in malloc argument
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 949da2db3e9357d8256863475a8c63409a8fc7b9
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 15:58:21 2022 -0700

    Fix -Wsign-compare warnings
    
    atom.c: In function ‘MakeAtom’:
    atom.c:161:65: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
                 if (hashTable[h]->hash == hash && hashTable[h]->len == len &&
                                                                     ^~
    atom.c:172:69: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
                     if (hashTable[h]->hash == hash && hashTable[h]->len == len &&
                                                                         ^~
    atom.c:207:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
         if (reverseMapSize <= a->atom) {
                            ^~
    bitmap.c: In function ‘bitmapGetMetrics’:
    bitmap.c:154:31: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
                 for (int i = 0; i < *glyphCount; i++) {
                                   ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit c0fb322b8d53e3f9fdffbba8c733c78ff3d50c85
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 15:50:53 2022 -0700

    Remove redundant null-pointer check
    
    As warned by cppcheck:
    
    bdfread.c:130:18: style: Condition '!line' is always false [knownConditionTrueFalse]
                if ((!line) || (bdfIsPrefix(line, "ENDCHAR")))
                     ^
    bdfread.c:126:13: note: Assuming condition '!line' is false
            if (!line)
                ^
    bdfread.c:130:18: note: Condition '!line' is always false
                if ((!line) || (bdfIsPrefix(line, "ENDCHAR")))
                     ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit d3e6a823537bb451be60febb6bfc769135fe5b35
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 15:46:57 2022 -0700

    Remove redundant assignment
    
    As warned by cppcheck:
    bdfutils.c:164:17: style: Variable 'pp' is reassigned a value before the old one has been used. [redundantAssignment]
            for (pp = s; *pp; pp++)
                    ^
    bdfutils.c:162:12: note: pp is assigned
            pp = s;
               ^
    bdfutils.c:164:17: note: pp is overwritten
            for (pp = s; *pp; pp++)
                    ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit fa8eed6deefbb96d9d3e2af1b23b9fff193bda92
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 15:41:31 2022 -0700

    Make some functions static that aren't called from other files
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 746899671c4d6f9d8b7fb25bb17109b331a1b2ed
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 15:23:36 2022 -0700

    Delete functions identified as unused by cppcheck
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

commit 6d526c90729d1b66da6c2546ad3ae794ff0f7751
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Fri Sep 16 14:34:41 2022 -0700

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