From 51fcacbb9f9b66946170159c974f0c7f350c4ea4 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <icculus@icculus.org>
Date: Thu, 13 Jan 2011 02:50:21 -0500
Subject: [PATCH] Added a FIXME.

---
 mojoshader_compiler.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/mojoshader_compiler.c b/mojoshader_compiler.c
index 3e225e7c..8ffcf8c3 100644
--- a/mojoshader_compiler.c
+++ b/mojoshader_compiler.c
@@ -2773,6 +2773,14 @@ static const MOJOSHADER_astDataType *type_check_ast(Context *ctx, void *_ast)
 
         case MOJOSHADER_AST_STRUCT_DECLARATION:
         {
+            // !!! FIXME: We don't handle struct predeclaration at all right now
+            // !!! FIXME:  (neither does the grammar)...not only does that mean
+            // !!! FIXME:  you need to know the struct definition up front, but
+            // !!! FIXME:  you can't do "struct XXX *next;" for a self-referencing
+            // !!! FIXME:  linked list struct thing. This probably isn't a big
+            // !!! FIXME:  deal, as there aren't (CURRENTLY!) pointers in HLSL,
+            // !!! FIXME:  but you never know.
+
             const MOJOSHADER_astStructMembers *mbrs;
 
             // !!! FIXME: count this during parsing?