<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: Fixes/improvements for -origpagesizes"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=72312#c19">Comment # 19</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - [patch] pdftops: Fixes/improvements for -origpagesizes"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=72312">bug 72312</a>
              from <span class="vcard"><a class="email" href="mailto:ajohnson@redneon.com" title="Adrian Johnson <ajohnson@redneon.com>"> <span class="fn">Adrian Johnson</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=72312#c15">comment #15</a>)
<span class="quote">> I am not sure whether the macro only does the equivalent of

> <</PageSize[595 842]/ImagingBBox null>>setpagedevice

> In this case we could simply skip its call when the paper size does not
> change (at least when I comment it out I get a correct printout and duplex
> works).</span >

That is basically all the macro does. It also checks if the setpagedevice
operator exists before calling it.

So yes we could just skip calling the macro when the page size does not change.
Alternatively I could change the macro to only change the page size if it is
different to the previous page. This has the advanatge of preserving page
independence as required by the DSC specification. ie the macro will be output
for every page so pages can be printed in any order and the correct page size
will be used but the macro will not alter the page size if the page size in the
page device dictionary is already correct.</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>