[Poppler-bugs] [Bug 20652] PDF reflow

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=20652





--- Comment #3 from Erik Hovland <erik at hovland.org>  2009-06-05 14:07:05 PST ---
Created an attachment (id=26474)
 --> (http://bugs.freedesktop.org/attachment.cgi?id=26474)
Adds -reflow to pdftohtml.

This patch is a rework of the patch that showed up in this email:
http://lists.freedesktop.org/archives/poppler/2008-September/004126.html

That patch was answered later by this email:
http://lists.freedesktop.org/archives/poppler/2009-January/004346.html

In that email some issues were brought up. This patch should address all of
those issues including:
1. The vertOverlap compiler warning is consequential to the reflow feature.
That part of the patch was separated out and attached to this bug.
2. The noMerge assignment in pdftohtml.cc when -xml is used has been removed.
3. Crufty unnecessary changes like the bgcolor and the comments on file
extensions have been removed from the patch.
4. The <p> markup now has corresponding </p> closure markup.

There are two extra changes in this patch that were not in the original posters
patch.
1. The file HtmlFonts.cc has a change to not do &nbsp for space if reflow is
on.
2. The original patch did not insert a space if merging a line. Some additional
logic was added to allow for that. I am especially interested in whether this
logic looks valid or whether it is faulty.

It is hoped that both the original bug reported and Alberto get a chance to try
out and review this patch. Thanks.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.