Here are two further patches to clean up the code:

* local_bidi.patch: The bidi object in the class Text is only used in places now where a ParagraphMetrics::computeRowMetrics call comes before. And this function the Bidi object is updated. So it is a little step now to remove the Text::bidi completely and use local Bidi objects instead without more work to do.

* cursor_get_font.patch: the Cursor::getFont is responsible for the cursor size. It should fit to the setCurrentFont function because the cursor tells the user which font will be used when typing characters. Especially on RTL/LTR boundaries it should fit because it is essential for the user to see which writing direction is active. This patch implements a simplified version of setCurrentFont's logic to archive this.

Stefan

Attachment: cursor_get_font.patch
Description: Binary data

Attachment: local_bidi.patch
Description: Binary data


Attachment: PGP.sig
Description: Signierter Teil der Nachricht

Reply via email to