[Poppler-bugs] [Bug 30692] [PATCH] pdftops does not crop

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=30692

--- Comment #13 from Albert Astals Cid <tsdgeos at terra.es> ---
(In reply to comment #12)
> Created attachment 70289 [details] [review]
> patch against the 19 Nov 12 git source
> 
> I made a version of the patch against 19 Nov 12 source.  The patch still
> works for me with the PDF that I attached previously.
> createGfx() calls makeBox() and then Gfx().
> makeBox() sometimes sets crop to FALSE, and then createGfx() does not pass a
> crop box to Gfx().
> The patch uses the original value of crop in the call to Gfx().
> Some files, like the one that I attached, draw outside the crop box, so even
> if the PDF sets a crop box of a given size, you still need to generate the
> postscript commands to crop to that size.

Right, this still does not make sense, you are creating a tempvalue so that the
returned value for makeBox is not used, if that is what you really want, why
just not remove that parameter?

On the other hand if this is about the PS conversion i do think that it is the
PSOutputDev that should be modified and for example cropping itself on the box
passed in the state of startPage ?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20121201/5a38f843/attachment.html>