<p dir="ltr">I believe some conf files have direct unicode (rather than escaped sequences) in them and that is preferred. </p>
<div class="gmail_quote">On 20 May 2014 23:28, "Jaak Ristioja" <<a href="mailto:jaak@ristioja.ee">jaak@ristioja.ee</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I've never done BiDi, but I'm not sure I need to take that into account<br>
while fixing the RTF parsing. As I currently understand it, this<br>
particular piece of code does not support any part from the RTF spec<br>
dealing with bidirectional text handling. Hence all BiDi information<br>
contained in the configuration file strings (e.g. About=) is contained<br>
either in the plain ASCII text or the \u<num> Unicode escapes which this<br>
algorithm should pass through unmodified.<br>
<br>
...except for HTML entities which should actually be escaped. This bug<br>
in the algorithm I previously failed to notice. Additionally I forgot<br>
that non-ASCII characters in the input string should also lead to<br>
parsing failure.<br>
<br>
Jaak<br>
<br>
<br>
On 20.05.2014 21:01, David Haslam wrote:<br>
> Take care with Right to Left languages such as Hebrew.<br>
><br>
> i.e. After any patches to the filter, please include some testing for BiDi<br>
> text in the About= field and others.<br>
><br>
> David<br>
><br>
><br>
><br>
> --<br>
> View this message in context: <a href="http://sword-dev.350566.n4.nabble.com/RTFHTML-filter-bugs-tp4653969p4653970.html" target="_blank">http://sword-dev.350566.n4.nabble.com/RTFHTML-filter-bugs-tp4653969p4653970.html</a><br>
> Sent from the SWORD Dev mailing list archive at Nabble.com.<br>
><br>
> _______________________________________________<br>
> sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
> <a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
> Instructions to unsubscribe/change your settings at above page<br>
><br>
<br>
<br>
<br>_______________________________________________<br>
sword-devel mailing list: <a href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/sword-devel" target="_blank">http://www.crosswire.org/mailman/listinfo/sword-devel</a><br>
Instructions to unsubscribe/change your settings at above page<br></blockquote></div>