[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

--- Comment #18 from alex <alexbodn.groups at gmail.com> ---
(In reply to comment #16)
> Sorry but no, our API is crap enough already to add stuff that you know you
> will remove later. If you don't need this enum, don't add it.
> 
> And if you need it tell me why, i still don't see this enum used anywhere.

i'm moving the enum into the TextPage object, to avoid polution and lots of
extra parameters.
the reordering mode parameter has a default value.
the patch will shrink greatly, and includes my fix for #2981 too.
it adds to the api nothing i'd remove in the future. only things i'd further
build upon: the reordering_mode option to the TextPage constructor.
yet a complete solution would require moving the reordering process to the core
of text loading.
here's the new patch.

-- 
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/20121128/5f975874/attachment-0001.html>