[PATCH] [Bug 91955] Bidi algorithm: incorrect implementation of rule N2

Mitz Pettel bugs.kde.org at mitzpettel.com
Wed Nov 3 23:46:18 GMT 2004


On Nov 3, 2004, at 10:20 PM, Stephan Kulow wrote:

> Am Mittwoch 03 November 2004 17:21 schrieb Mitz Pettel:
>> I see. I think changing line 859 from
>>                       if( bidi.status.eor != QChar::DirL ) {
>> to
>>                       if( bidi.status.eor != QChar::DirL &&
>> bidi.status.eor!=QChar::DirEN ) {
>> should take care of most of these.
>
> I will try tomorrow. Thanks for the fix.

Forget it. I tried it myself and it failed one of the test cases. 
Here's the latest revision of the patch. This one looks good on 
bidi.html, and should not generate excessive runs (however I didn't 
test that part).

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bidi.cpp.diff
Type: application/octet-stream
Size: 4305 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20041104/28f50e19/attachment.obj>
-------------- next part --------------


Best regards,
-- Mitz Pettel


More information about the kfm-devel mailing list