<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
Hi,<br>I am almost finished updating my "old" patches into the new version of pdftops.<br>The new OverprintPreview code interferes with it in PSOutput.cc, and I suspect that is why the patches were dropped.<br>The old patches are actually three sets of patches: part of the implementation of -binary, a bug fix where level2sep and level3sep wrote rgb color instead of cmyk color, and an optimization to write color images as mono if they contain only shades of gray.<br>I am redoing my old patches to use a single case block instead treating level2 and level3 in a different case block than level2sep and level3sep to avoid repeating code.<br>I still have all of the test files and scripts from when I wrote the original patches, so I can test the new patches well before I post them.<br>William<br><br><div><div id="SkyDrivePlaceholder"></div>&gt; From: aacid@kde.org<br>&gt; To: poppler@lists.freedesktop.org<br>&gt; Date: Fri, 24 Feb 2012 19:12:02 +0100<br>&gt; Subject: Re: [poppler] poppler pdftops regressions<br>&gt; <br>&gt; El Divendres, 24 de febrer de 2012, a les 09:33:03, Thomas Freitag va <br>&gt; escriure:<br>&gt; &gt; Am 23.02.2012 23:18, schrieb Albert Astals Cid:<br>&gt; &gt; &gt; El Dijous, 23 de febrer de 2012, a les 14:36:29, William Bader va <br>&gt; escriure:<br>&gt; &gt; &gt;&gt; Hi,<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; Hi<br>&gt; &gt; &gt; <br>&gt; &gt; &gt;&gt; I am using a 32-bit i686 version of Fedora 14 Linux.<br>&gt; &gt; &gt;&gt; I copied poppler from git earlier today (Feb 23, 2012), and noticed<br>&gt; &gt; &gt;&gt; two<br>&gt; &gt; &gt;&gt; problems.<br>&gt; &gt; &gt;&gt; <br>&gt; &gt; &gt;&gt; pdftops -level1 does not seem to handle bitmaps well.<br>&gt; &gt; &gt;&gt; My test files 020_ReadMe_Ghent_Output_Patch.pdf (Ghent Output Suite<br>&gt; &gt; &gt;&gt; document), bug-poppler27482-precision.pdf (Herbstfest bei MEIFORT ad),<br>&gt; &gt; &gt;&gt; elp-B83378-gray.pdf, and testimagemask.pdf (real estate ad with photos<br>&gt; &gt; &gt;&gt; of six agents at the bottom) showed like checkerboards. The size is<br>&gt; &gt; &gt;&gt; what I expect, and they don't get errors.  I suspect that the problem<br>&gt; &gt; &gt;&gt; is a change in the image code in the pdftops prolog.<br>&gt; &gt; &gt;&gt; <br>&gt; &gt; &gt;&gt; pdftops -binary no longer has much effect.<br>&gt; &gt; &gt;&gt; It is present PSOutput.cc in the recent tarball release of 0.18.4, but<br>&gt; &gt; &gt;&gt; about half of the code is missing in the git version. Was that<br>&gt; &gt; &gt;&gt; intentional, or did a patch get applied to a branch that was not<br>&gt; &gt; &gt;&gt; later merged?<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; The patch was merged correctly but Thomas smashed your changes when<br>&gt; &gt; &gt; merging xpdf303 changes and we did not realize<br>&gt; &gt; <br>&gt; &gt; Sorry if I missed something. I'll have a look at it at the weekend, I'm<br>&gt; &gt; anyway working on a patch for CJK fonts in PSOutput. Are the "old"<br>&gt; &gt; patches from William available? <br>&gt; <br>&gt; Sure https://bugs.freedesktop.org/show_bug.cgi?id=34003 or <br>&gt; http://cgit.freedesktop.org/poppler/poppler/commit/?id=c3470145f95791167c19a438934a923eab8a93cf<br>&gt; <br>&gt; &gt; It's probably easier to look into the<br>&gt; &gt; patches and see what is missing now (of course together with comparing<br>&gt; &gt; code between tarball and git). Also it would be nice to have<br>&gt; &gt; elp-B83378-gray.pdf and testimagemask.pdf for testing, the other I have.<br>&gt; &gt; Didn't think that still someone use level1 option :-) Aren't that<br>&gt; &gt; printers been already in a museum?<br>&gt; <br>&gt; William sent a patch for the level1 image regressionwhich i'm testing at the <br>&gt; moment.<br>&gt; <br>&gt; Albert<br>&gt; <br>&gt; &gt; <br>&gt; &gt; Thomas<br>&gt; &gt; <br>&gt; &gt; &gt;&gt; Should I open bug reports for each of these issues?<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; Please, and if you can give patches to fix them even better :-)<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; Albert<br>&gt; &gt; &gt; <br>&gt; &gt; &gt;&gt; William<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; _______________________________________________<br>&gt; &gt; &gt; poppler mailing list<br>&gt; &gt; &gt; poppler@lists.freedesktop.org<br>&gt; &gt; &gt; http://lists.freedesktop.org/mailman/listinfo/poppler<br>&gt; &gt; &gt; <br>&gt; &gt; &gt; .<br>&gt; &gt; <br>&gt; &gt; _______________________________________________<br>&gt; &gt; poppler mailing list<br>&gt; &gt; poppler@lists.freedesktop.org<br>&gt; &gt; http://lists.freedesktop.org/mailman/listinfo/poppler<br>&gt; _______________________________________________<br>&gt; poppler mailing list<br>&gt; poppler@lists.freedesktop.org<br>&gt; http://lists.freedesktop.org/mailman/listinfo/poppler<br></div>                                               </div></body>
</html>