Bug 27290 - Incorrect '-zoom' handling in pdftohtml
Summary: Incorrect '-zoom' handling in pdftohtml
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
Hardware: All All
: medium normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-24 07:39 UTC by Pavel Denisov
Modified: 2010-12-23 12:28 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Patch (901 bytes, patch)
2010-03-24 07:39 UTC, Pavel Denisov
Details | Splinter Review
PDF that can be used for testing (582.40 KB, application/pdf)
2010-03-24 07:40 UTC, Pavel Denisov
Details

Description Pavel Denisov 2010-03-24 07:39:48 UTC
Created attachment 34407 [details] [review]
Patch

pdftohtml rotates background instead of page scaling when '-zoom' option > 1.
Comment 1 Pavel Denisov 2010-03-24 07:40:55 UTC
Created attachment 34408 [details]
PDF that can be used for testing
Comment 2 Pavel Denisov 2010-03-24 08:03:24 UTC
Comment on attachment 34408 [details]
PDF that can be used for testing

pdftohtml -nomerge -noframes -c test.pdf
Comment 3 Pavel Denisov 2010-03-24 08:04:24 UTC
Command to test:

pdftohtml -nomerge -noframes -c test.pdf
Comment 4 Albert Astals Cid 2010-04-24 06:27:58 UTC
Sorry we can not accept the patch, while it is true that it fixes this very particular file, it causes a regression when running "pdftohtml 2006rawsurveyreport.pdf" changing

We would also like to thank the 400+ members of the&nbsp;<a href="http://openraw.org/lists/">OpenRAW mailing list</a>

to

<a href="http://openraw.org/lists/">We would also like to thank the 400+ members of the&nbsp;OpenRAW mailing list</a>

We'd be more than happy that accept the patch if it didn't cause regressions.

http://www.openraw.org/files/2006rawsurveyreport.pdf
Comment 5 Pavel Denisov 2010-12-21 03:38:18 UTC
The bug doesn't exist in 0.15.3 anymore.
Comment 6 Petr Baudis 2010-12-21 18:37:21 UTC
I still need the patch for poppler-0.15.3 for -zoom to work at all. However, only the first hunk is required; the second affects only dpi of the pdf, which should be mostly irrelevant.

Albert: This regression happens *only* because -zoom actually starts to work with this patch, and its default value is not 1, but 1.5. Therefore, with this patch, all PDFs are suddenly zoomed to 1.5x and it appears there is an orthogonal bug somewhere in the link detection (with -zoom 2, the link is correct again). I think:

* such a small regression seems incomparable with actually working -zoom option, which can produce so much better PDFs to read, especially on larger resolutions

* if you still care, simply change scale default value from 1 to 1.5. This will mean that by default, the output shall be pretty much identical as in previous versions, but users will actually be able to change the zoom settings if they want.
Comment 7 Albert Astals Cid 2010-12-23 12:28:11 UTC
This patch has been commited meanwhile due to some other reason and well, zooming to 1.5 was the default in the --help and in the man page so i think it's time we honour it :D

So yeah it will change the output from 0.14 to 0.16 but it will do what it was supposed to do do i think i'll leave it that way.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.