diff options
author | Robin Watts <robin.watts@artifex.com> | 2012-01-23 11:23:01 +0000 |
---|---|---|
committer | Robin Watts <robin.watts@artifex.com> | 2012-01-23 11:23:01 +0000 |
commit | 03611f8dd4b349042e0edc29ea43f84d32f5601c (patch) | |
tree | 21048737a0ac06b2e32519b6ae3c41b3ed602518 /draw | |
parent | 28afe8780af402b99efa6b646ef96f1a777bfb11 (diff) | |
download | mupdf-03611f8dd4b349042e0edc29ea43f84d32f5601c.tar.xz |
Remove assert in tiling code.
Both Zeniko and Malc have said that this causes problems.
Following a hint from malc, the following command causes the
assert to trigger:
win32/Debug/pdfdraw.exe -r0.3 -o out.png ../MyTests/pdf_reference17.pdf 1143
I suspect it's because of the extreme resolution, and the round_rect
call. At any rate, disabling the assert is of no real import.
Diffstat (limited to 'draw')
-rw-r--r-- | draw/draw_device.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/draw/draw_device.c b/draw/draw_device.c index 12810420..89169ab6 100644 --- a/draw/draw_device.c +++ b/draw/draw_device.c @@ -1414,9 +1414,11 @@ fz_draw_begin_tile(fz_device *devp, fz_rect area, fz_rect view, float xstep, flo bbox = fz_round_rect(fz_transform_rect(ctm, view)); /* We should never have a bbox that entirely covers our destination. * If we do, then the check for only 1 tile being visible above has - * failed. */ - assert(bbox.x0 > state->dest->x || bbox.x1 < state->dest->x + state->dest->w || - bbox.y0 > state->dest->y || bbox.y1 < state->dest->y + state->dest->h); + * failed. Actually, this *can* fail due to the round_rect, at extreme + * resolutions, so disable this assert. + * assert(bbox.x0 > state->dest->x || bbox.x1 < state->dest->x + state->dest->w || + * bbox.y0 > state->dest->y || bbox.y1 < state->dest->y + state->dest->h); + */ dest = fz_new_pixmap_with_rect(dev->ctx, model, bbox); fz_clear_pixmap(dest); shape = state[0].shape; |