<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - [PATCH] try to detect line breaks in the PDF and insert them in raw mode for pdftotext" href="https://bugs.freedesktop.org/show_bug.cgi?id=62266#c10">Comment # 10</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - [PATCH] try to detect line breaks in the PDF and insert them in raw mode for pdftotext" href="https://bugs.freedesktop.org/show_bug.cgi?id=62266">bug 62266</a> from <a class="email" href="mailto:aacid@kde.org" title="Albert Astals Cid <aacid@kde.org>"> Albert Astals Cid</a> <pre>(In reply to <a href="show_bug.cgi?id=62266#c9">comment #9</a>) > > it is just an assumption that if two characters are separated enough one from the other, there is a space in the middle > > It is more than that. As I said: > > > For example, the current code inserts a new line whenever the next word is detected to not be in the same line as the current word > > The raw text isn't just having spaces added, but it is also getting new > lines added whenever the vertical space between the current word and the > next word exceeds the `maxIntraLineDelta` constant. > > My patch is a very small extension of this sort of logic: add an additional > new line when the vertical space between the current word and next word > exceeds the `maxLineSpacingDelta` constant. > > I don't think my patch makes any additional assumptions beyond the > assumptions already made by the code. It may not, but i don't see the need for your patch (you haven't made a case for it) and more code means more code I need to maitain for the rest of my life. In my opinion you are trying to use raworder for something that raworder is not supposed to do, why are you using raw order instead of the real physical order?</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>