diff options
author | Ken Sharp <ken.sharp@artifex.com> | 2018-02-06 11:46:56 +0000 |
---|---|---|
committer | Ken Sharp <ken.sharp@artifex.com> | 2018-02-06 11:46:56 +0000 |
commit | f6005484617ada241d2f5f4deee4808616474831 (patch) | |
tree | 47e31609e0c7d6b161621748e9fedbba61c0d90b /base/gstype2.c | |
parent | 16688a8b833cf4e8950265e933604c32b1f2392e (diff) | |
download | ghostpdl-f6005484617ada241d2f5f4deee4808616474831.tar.gz |
Address more scanbuild/gcc/clang warnings
gdevkrnlsclass.c Change a while...do into a do...while to see if it
pacifies a 'dereference of NULL pointer' in scanbuild
gsparamx.c Incorrect return code could lead to an earlier error not
being preserved. Fortunately this is currently only called by claptrap.
gsptype1.c remove unnecessary assignments
gssprintf.c " " "
gstype2.c " " "
gstype42.c " " "
gxfill.c " " "
gsptype2.c rejig the code to avoid setting a return code that isn't used.
Add a check on a return code that was missing.
gxchar.c add return code checking
gxcmap.c " " " "
Diffstat (limited to 'base/gstype2.c')
-rw-r--r-- | base/gstype2.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/base/gstype2.c b/base/gstype2.c index e51d0fe8d..70dec6fa2 100644 --- a/base/gstype2.c +++ b/base/gstype2.c @@ -778,7 +778,6 @@ flex: { } if (code < 0) return code; - csp -= 13; } cnext; case ce2_hflex1: |