diff options
| author | Robin Watts <robin.watts@artifex.com> | 2012-07-18 16:01:13 +0100 |
|---|---|---|
| committer | Robin Watts <robin.watts@artifex.com> | 2012-07-18 19:00:01 +0100 |
| commit | 71340325936893d7ea993bf0ff78e5ab5b695fd0 (patch) | |
| tree | 807b35370faa78f7be545feebe233252842ece15 /draw | |
| parent | aa64c27048e9d308400058b40ee2677cfd7c5081 (diff) | |
| download | mupdf-71340325936893d7ea993bf0ff78e5ab5b695fd0.tar.xz | |
Remove pointless test.
Variable i can never be zero at this point. The desired point of testing
against i was to ensure that the test did not evaluate true at the
first run, and the other parts of the condition are sufficent to
ensure this, so just remove the test on i.
Diffstat (limited to 'draw')
| -rw-r--r-- | draw/draw_path.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/draw/draw_path.c b/draw/draw_path.c index b0889279..9996c182 100644 --- a/draw/draw_path.c +++ b/draw/draw_path.c @@ -82,7 +82,7 @@ fz_flatten_fill_path(fz_gel *gel, fz_path *path, fz_matrix ctm, float flatness) { case FZ_MOVETO: /* implicit closepath before moveto */ - if (i && (cx != bx || cy != by)) + if (cx != bx || cy != by) line(gel, &ctm, cx, cy, bx, by); x1 = path->items[i++].v; y1 = path->items[i++].v; @@ -118,7 +118,7 @@ fz_flatten_fill_path(fz_gel *gel, fz_path *path, fz_matrix ctm, float flatness) } } - if (i && (cx != bx || cy != by)) + if (cx != bx || cy != by) line(gel, &ctm, cx, cy, bx, by); } |