Partially revert 86c68c7a2c971efe8e35b1f1bd99401dc8b688d2
authorBehdad Esfahbod <behdad@behdad.org>
Wed, 24 Feb 2016 06:53:40 +0000 (15:53 +0900)
committerBehdad Esfahbod <behdad@behdad.org>
Wed, 24 Feb 2016 06:53:40 +0000 (15:53 +0900)
commitebd7431f824c718db6ce5b85e94f2b3911127af4
tree76619ce45f2b393209132d189b2d227e42a64fbc
parent284481b3120963353f5f34ed094f07ffc13480ef
Partially revert 86c68c7a2c971efe8e35b1f1bd99401dc8b688d2

That commit moved the advance adjustment for mark positioning to
be applied immediately, instead of doing late before.  This breaks
if mark advances are zeroed late, like in Arabic.  Also, easier to
hit it in RTL scripts since a single mark with non-zero advance is
enough to hit the bug, whereas in LTR, at least two marks are needed.

This reopens https://github.com/behdad/harfbuzz/issues/211
The cursive+mark interaction is broken again.  To be fixed in a
different way.
src/hb-ot-layout-gpos-table.hh
test/shaping/tests/cursive-positioning.tests