<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [PATCH] pdftops does not crop"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=30692#c17">Comment # 17</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [PATCH] pdftops does not crop"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=30692">bug 30692</a>
              from <span class="vcard"><a class="email" href="mailto:tsdgeos@terra.es" title="Albert Astals Cid <tsdgeos@terra.es>"> <span class="fn">Albert Astals Cid</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=30692#c16">comment #16</a>)
<span class="quote">> >The issue as I see it is that you give the PSOutputDev a box, and it allows commands outside that box to be rendered, that is wrong and shall be fixed at the PSOutputDev level, not somewhere else.

> With the patch, pdftops crops by generating a single crop box in postscript
> at the beginning of the page.  I think that xpdf (which works) does that
> also.  Checking the crop box inside poppler before rendering each object
> could be expensive, and it could be tricky with text that was part inside
> and part outside.</span >

I never suggested checking the for each object. I suggested that
PSOutputDev::startPage generates the cropbox with the given box in the state

<span class="quote">> 
> In theory, a pdf should not attempt to draw much outside the crop box other
> than a small amount of descriptive information (as in the example pdf that I
> attached) or registration marks for printing.  The example pdf has an image
> of an ad that was intended to be placed on a newspaper page.  The crop box
> marks the part of the image that should appear on the page.  The objects
> outside the crop box are for proofing the image as a stand-alone item.</span >

Yes I know what a crop box is for.

<span class="quote">> I have the same problem as you that I have very little time.  If you can
> tell me exactly where poppler should make the crop test, I can try it.  The
> patch has been working for me for years, and one of the libreoffice people
> said that it worked for them also, and it is only a very small change. 
> Isn't a bigger change more likely risk side-effects that could take a while
> to discover?</span >

Because to me the patch looks like a hack, yes, hacks work, but they are not
proper fixes.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>