<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:williambader@hotmail.com" title="William Bader <williambader@hotmail.com>"> <span class="fn">William Bader</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Printing of certain PDF files does not work with "fit-to-page" because of wrong BoundingBox values in the PostScript"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87161">bug 87161</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>williambader@hotmail.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Printing of certain PDF files does not work with "fit-to-page" because of wrong BoundingBox values in the PostScript"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87161#c12">Comment # 12</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Printing of certain PDF files does not work with "fit-to-page" because of wrong BoundingBox values in the PostScript"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87161">bug 87161</a>
from <span class="vcard"><a class="email" href="mailto:williambader@hotmail.com" title="William Bader <williambader@hotmail.com>"> <span class="fn">William Bader</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=119266" name="attach_119266" title="alternate patch by Adrian Johnson">attachment 119266</a> <a href="attachment.cgi?id=119266&action=edit" title="alternate patch by Adrian Johnson">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=87161&attachment=119266'>[review]</a>
alternate patch by Adrian Johnson
This is an alternate patch submitted to the poppler mailing list by Adrian
Johnson on Oct 27. He added the text below.
==========
I've reviewed the patch and have the following comments.
It would have been a lot easier to review (and probably would have been
reviewed earlier) if you avoided the unnecessary changes to convert
if-else statements to case statements. Putting code style changes in a
separate patch to the bug fix makes reviewing changes much easier.
+ int(pbbty),
+ int(width * xScale + pbbtx + 0.5),
Using floor() and ceil() would be better and would make the code easier
to understand.
You appear to be ignoring the value of tx and ty prior to centering
calculations. What happens if tx and ty are non zero at this point? I
would be more comfortable with the patch if the page bbox calculations
used the exact same transformation as is output to PS.
I'm attaching a new patch that I think is a lot easier to understand. It
uses the same transformation as is output to PS to calculate the
bounding box.</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>