diff --git a/tools/qfcc/include/def.h b/tools/qfcc/include/def.h
index d538d111d..79502a7d0 100644
--- a/tools/qfcc/include/def.h
+++ b/tools/qfcc/include/def.h
@@ -222,7 +222,9 @@ void def_to_ddef (def_t *def, ddef_t *ddef, int aux);
 
 /** Initialize a def referenced by the given symbol.
 
-	The symbol is checked for redefinition. (FIXME check rules)
+	The symbol is checked for redefinition. A symbol is considered to be
+	redefined if the previous definition is in the same symbol table and
+	of a different type or already initialized.
 
 	If \a type is null, then the def will be given the default type (as
 	specified by ::type_default).
diff --git a/tools/qfcc/include/expr.h b/tools/qfcc/include/expr.h
index 08614f2ce..f1987bb8a 100644
--- a/tools/qfcc/include/expr.h
+++ b/tools/qfcc/include/expr.h
@@ -416,7 +416,7 @@ expr_t *new_entity_expr (int entity_val);
 
 /** Create a new field constant expression node.
 
-	\param field_val	XXX
+	\param field_val	offset? XXX
 	\param type		The type of the field.
 	\param def
 	\return			The new field constant expression node
diff --git a/tools/qfcc/include/function.h b/tools/qfcc/include/function.h
index a1a5c19bf..b4b546fa6 100644
--- a/tools/qfcc/include/function.h
+++ b/tools/qfcc/include/function.h
@@ -105,9 +105,9 @@ extern function_t *current_func;
 typedef struct param_s {
 	struct param_s *next;
 	const char *selector;
-	struct type_s *type;		//FIXME redundant
-	const char *name;			//FIXME redundant
-	struct symbol_s *symbol;
+	struct type_s *type;
+	const char *name;
+	struct symbol_s *symbol;	//FIXME what is this for?
 } param_t;
 
 struct expr_s;