From f83f0f4836691b04306c2ef80979f2e1d76a2f28 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Mon, 19 Sep 2011 18:51:48 -0400
Subject: [PATCH] [graphite] Add note about graphite shaker brokenness

---
 src/hb-graphite2.cc | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/hb-graphite2.cc b/src/hb-graphite2.cc
index a3dc1b455..067575968 100644
--- a/src/hb-graphite2.cc
+++ b/src/hb-graphite2.cc
@@ -329,6 +329,14 @@ hb_graphite_shape (hb_font_t          *font,
     curradvy += pPos->y_advance;
   }
   pPos[-1].x_advance += gr_seg_advance_X(seg) - curradvx;
+
+  /* TODO(behdad):
+   * This shaper is badly broken with RTL text.  It returns glyphs
+   * in the logical order!
+   */
+//  if (HB_DIRECTION_IS_BACKWARD (buffer->props.direction))
+//    hb_buffer_reverse (buffer);
+
   success = 1;
 
 dieout: