summaryrefslogtreecommitdiff
path: root/draw
diff options
context:
space:
mode:
authorRobin Watts <robin.watts@artifex.com>2012-01-23 11:23:01 +0000
committerRobin Watts <robin.watts@artifex.com>2012-01-23 11:23:01 +0000
commit03611f8dd4b349042e0edc29ea43f84d32f5601c (patch)
tree21048737a0ac06b2e32519b6ae3c41b3ed602518 /draw
parent28afe8780af402b99efa6b646ef96f1a777bfb11 (diff)
downloadmupdf-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.c8
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;