[webkit-reviews] review requested: [Bug 4844] Render "simple" Hebrew using the CG codepath : [Attachment 3768] Exclude Hebrew letters and maqaf from ATSUI rendering

[bugzilla-request-daemon at opendarwin.org]

opendarwin.org at mitzpettel.com has asked Darin Adler <darin at apple.com> for
review:
Bug 4844: Render "simple" Hebrew using the CG codepath
http://bugzilla.opendarwin.org/show_bug.cgi?id=4844

Attachment 3768: Exclude Hebrew letters and maqaf from ATSUI rendering
http://bugzilla.opendarwin.org/attachment.cgi?id=3768&action=edit

------- Additional Comments from opendarwin.org at mitzpettel.com
> OK. Issue (1) is not a concern, but issue (2) is. I think we should put this
on
> hold again until that's dealt with.

I think the exact opposite in this case. The first time I pulled the patch was
because it made things worse for lines mixing "simple" Hebrew with non-"simple"
Hebrew (say, one word with niqqud), since they went from all-ATSU lines to
mixed ATSU/CG, and there was a problem with mixed ATSU/CG.

Issue (2) currently affects (certain) mixed Latin/Hebrew lines, since they
currently mix ATSU with CG, and there is (still) a problem with mixed ATSU/CG,
but this time, changing Hebrew to CG will actually make things better for these
lines.