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/gxfill.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/gxfill.c')
-rw-r--r-- | base/gxfill.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/base/gxfill.c b/base/gxfill.c index a332ff123..d07804c30 100644 --- a/base/gxfill.c +++ b/base/gxfill.c @@ -1892,7 +1892,6 @@ intersect_al(line_list *ll, fixed y, fixed *y_top, int draw, bool all_bands) nx = nx0; } endp->x_next = alp->x_next = nx; /* Ensure same X. */ - draw = 0; /* Can't guarantee same x for triple intersections here. Will take care below */ } |