<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - possibly incorrect calculation of indexes of PopplerTextAttributes" href="https://bugs.freedesktop.org/show_bug.cgi?id=54504#c1">Comment # 1</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - possibly incorrect calculation of indexes of PopplerTextAttributes" href="https://bugs.freedesktop.org/show_bug.cgi?id=54504">bug 54504</a> from <a class="email" href="mailto:jason@aquaticape.us" title="Jason Crain <jason@aquaticape.us>"> Jason Crain</a> <pre>Created <a href="attachment.cgi?id=70475" name="attach_70475" title="check if words end with spaces">attachment 70475</a> <a href="attachment.cgi?id=70475&action=edit" title="check if words end with spaces">[details]</a> <a href='page.cgi?id=splinter.html&bug=54504&attachment=70475'>[review]</a> check if words end with spaces poppler_page_get_text_layout and poppler_page_get_text_attributes assume that each word ends with a space or newline, causing them to become mismatched from the text. This patch adds a check to TextWord::getSpaceAfter. This fixes the problem for this file, but there are still other situations where the indexes can become mismatched because of the way TextSelectionDumper::getText aligns tables. I don't know how to fix that unless you want to modify poppler_page_get_text to return something simpler, instead of calling TextPage::getSelectionText to get the physical layout.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>