diff --git a/src/hb-ot-layout-private.h b/src/hb-ot-layout-private.h
index 48e9145dcd1926e992aa6fa06a2a81fadb54dd05..3fdd428393db0f7e04de8dfce2fc892a8bf210a7 100644
--- a/src/hb-ot-layout-private.h
+++ b/src/hb-ot-layout-private.h
@@ -28,7 +28,9 @@
 #define HB_OT_LAYOUT_PRIVATE_H
 
 #include "hb-private.h"
+
 #include "hb-ot-layout.h"
+
 #include "hb-buffer-private.h"
 
 
diff --git a/src/hb-ot-layout.cc b/src/hb-ot-layout.cc
index d04d54d979e3c1b401142ebb19c101f19873bd02..7e0539a6cff08253c59289454336e622578586b9 100644
--- a/src/hb-ot-layout.cc
+++ b/src/hb-ot-layout.cc
@@ -28,10 +28,6 @@
 
 #define HB_OT_LAYOUT_CC
 
-/* XXX */
-#include "hb-buffer-private.h"
-
-#include "hb-ot-layout.h"
 #include "hb-ot-layout-private.h"
 
 #include "hb-ot-layout-open-private.h"
@@ -263,7 +259,7 @@ hb_ot_layout_set_glyph_class (hb_ot_layout_t             *layout,
 			      hb_codepoint_t              glyph,
 			      hb_ot_layout_glyph_class_t  klass)
 {
-  /* TODO optimize this, similar to old harfbuzz code for example */
+  /* TODO optimize this? similar to old harfbuzz code for example */
 
   hb_ot_layout_class_t gdef_klass;
   int len = layout->new_gdef.len;
@@ -271,6 +267,7 @@ hb_ot_layout_set_glyph_class (hb_ot_layout_t             *layout,
   if (HB_UNLIKELY (glyph > 65535))
     return;
 
+  /* XXX this is not threadsafe */
   if (glyph >= len) {
     int new_len;
     unsigned char *new_klasses;