Re: [patch] Re: RTL justification bug

[Stefan Schimanski]

+       // If the last logical character is a separator, skip it, unless
+       // it's in the last row of a paragraph
+       if (end > 0 && end < par.size() && par.isSeparator(end - 1))
+               skipped_sep_vpos = bidi.log2vis(end - 1);

  HEBREW[ english]

rendered as

  [ english]WERBEH

No?

I thought it's about the visually last (or first for rtl paragraphs) character. Take

+ // If the last logical character is a separator, don't paint it, unless

+       // it's in the last row of a paragraph
+       if (end > 0 && end < par_.size() && par_.isSeparator(end - 1))
+               skipped_sep_vpos = bidi_.log2vis(end - 1);
+       

Same here.

Stefan

PGP.sig
Description: Signierter Teil der Nachricht