[Poppler-bugs] [Bug 55977] handling of rtl text inversion is too naive

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=55977

Adam Reichold <adam.reichold at t-online.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |adam.reichold at t-online.de

--- Comment #49 from Adam Reichold <adam.reichold at t-online.de> ---
Created attachment 117108
  --> https://bugs.freedesktop.org/attachment.cgi?id=117108&action=edit
Patch to use ICU for RTL text reordering

Hello,

I was made aware of this bug/patch by an RTL-using acquaintance and would like
to try to help to get this into shape for inclusion into master. I took
attachment 88871 and tried to clean it up manually, making sure that CMake and
autotools build systems work properly, that the current code paths stay intact
and that the new conditional compilation is easy to follow.

I did not yet do any functional testing beyond running the usual unit tests. I
think extending those with a dedicated RTL test case seems in order? I will try
to get help some from my acquaintance on that.

I did also remove the extensions to pdftotext and the reordering name parsing
in GlobalParams to keep the patch as short as possible for now.

Best regards, Adam.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20150714/958eb964/attachment.html>