Added a FIXME.

icculus · Jan 13, 2011 · 51fcacb · 51fcacb
1 parent 372e723
commit 51fcacb
Showing 1 changed file with 8 additions and 0 deletions.
diff --git 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?