http://bugs.winehq.org/show_bug.cgi?id=7150
--- Comment #12 from Shachar Shemesh shachar@shemesh.biz 2009-10-26 03:00:44 --- (In reply to comment #9)
Created an attachment (id=23239)
--> (http://bugs.winehq.org/attachment.cgi?id=23239) [details]
faster as it skips shaping if there is nothing to shape
I don't think what this comment should prevent this patch from going in, but better handle these things earlier than later.
The proper place for shaping to take place is before reordering, not after it. The proper BiDi handling is: - Bidi level calculations - Shaping - Line breaking - Reordering
I know that, with the current implementation, level calculation and reordering are done in one step, and that therefor it is a little impossible to implement the steps as described, but this is, non the less, where we should aspire to be.
More details at http://www.lingnu.com/en/programming.html (plug warning - articles written by me).
Shachar