<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:adam.reichold@t-online.de" title="Adam Reichold <adam.reichold@t-online.de>"> <span class="fn">Adam Reichold</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - handling of rtl text inversion is too naive"
href="https://bugs.freedesktop.org/show_bug.cgi?id=55977">bug 55977</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">CC</td>
<td>
</td>
<td>adam.reichold@t-online.de
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - handling of rtl text inversion is too naive"
href="https://bugs.freedesktop.org/show_bug.cgi?id=55977#c49">Comment # 49</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - handling of rtl text inversion is too naive"
href="https://bugs.freedesktop.org/show_bug.cgi?id=55977">bug 55977</a>
from <span class="vcard"><a class="email" href="mailto:adam.reichold@t-online.de" title="Adam Reichold <adam.reichold@t-online.de>"> <span class="fn">Adam Reichold</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=117108" name="attach_117108" title="Patch to use ICU for RTL text reordering">attachment 117108</a> <a href="attachment.cgi?id=117108&action=edit" title="Patch to use ICU for RTL text reordering">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=55977&attachment=117108'>[review]</a>
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
<span class=""><a href="attachment.cgi?id=88871" name="attach_88871" title="Alex patch rebased">attachment 88871</a> <a href="attachment.cgi?id=88871&action=edit" title="Alex patch rebased">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=55977&attachment=88871'>[review]</a> 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.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>