On Tue, 1 May 2018 10:21:55 +0200 Andreas Bilke wrote: > I think this patch opens a new bug (see my previous post) and should not > count as a bug fix for this issue.
Dear Andreas, I fully acknowledge that my single line patch is not a proper fix for the regression. I mean: it eliminates the poor-raster-image-rendering regression, but introduces a regression in the support for notes from PDF annotations (which, as far as I can tell, is a more recent feature). I fully agree with you that a proper fix for the poor-raster-image-rendering regression should fix the issue, without introducing other regressions. Maybe this requires improving the Poppler library: have you considered getting in touch with the Poppler developers? However I think this temporary patch should be kept in the Debian package (and possibly even adopted in the upstream pdfpc project) until a proper fix is devised and implemented. A newer feature (notes from PDF annotations) should not be introduced at the cost of breaking an older one (good raster image rendering). Hence, I think it's better if pdfpc makes a little step back (temporarily losing support for notes from PDF annotations) in order to keep a good raster image rendering capability. I hope this reasoning makes sense to you. Thanks for your helpfulness! P.S. for Barak: you have probably already read this in my long reasoning above, but let me say it explicitly: I can confirm that pdf-presenter-console/4.1-4 eliminates the poor-raster-image-rendering regression; thanks for uploading a new Debian revision to unstable so promptly! -- http://www.inventati.org/frx/ There's not a second to spare! To the laboratory! ..................................................... Francesco Poli . GnuPG key fpr == CA01 1147 9CD2 EFDF FB82 3925 3E1C 27E1 1F69 BFFE
pgpK7pY5_NrcC.pgp
Description: PGP signature
