[PATCH] [Bug 91955] Bidi algorithm: incorrect implementation of rule N2
Stephan Kulow
coolo at kde.org
Thu Nov 4 10:15:33 GMT 2004
Am Wednesday 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.
>
That patch surely reduced the number of false hits.
Your patch also changes the rendering of the test case
to bug 64506, which now renders exactly as in firefox.
It also improves the rendering of test case to 62904, so I would
say the patch should go in. But I await a second oppinion.
BTW: I put the key word bidi in all bugs reports, so you can easier
query for them in case you feel like continue touching this code ;)
Greetings, Stephan
More information about the kfm-devel
mailing list