<table><tr><td style="">michaelweghorn updated this revision to Diff 49518.<br />michaelweghorn added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18179">View Revision</a></tr></table><br /><div><div><p>Implement improvements suggested by Albert:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">use an enum rather than a bool for scale mode</li>
<li class="remarkup-list-item">avoid ABI breakage by adding extra methods that take an additional parameter for scale mode and make existing methods call those</li>
</ul>

<p>What's a bit ugly here is that 'scaleMode' cannot be the last<br />
parameter in the new 'FilePrinter::printFile' method, as it<br />
has to come before those params that have a default value.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D18179?vs=49225&id=49518">https://phabricator.kde.org/D18179?vs=49225&id=49518</a></div></div><br /><div><strong>BRANCH</strong><div><div>michaelweghorn/UPDATE_20191115_D18179_allowScaledPrintingWithoutRasterization</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18179">https://phabricator.kde.org/D18179</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>core/fileprinter.cpp<br />
core/fileprinter.h<br />
generators/poppler/generator_pdf.cpp</div></div></div><br /><div><strong>To: </strong>michaelweghorn, Okular, ngraham, sander<br /><strong>Cc: </strong>aacid, fvogt, okular-devel, tfella, ngraham, darcyshen<br /></div>