X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Fvalidate.cpp;h=b8710509e8d32fe424062f6c6558976ceab6da35;hb=3f44e477f81983c66947fe8a6d8640a3b2f9e0b3;hp=13cb5ccb23ad15e4d6e56798552ffcd35fefd381;hpb=6cb4cc995865935d22b276801c218d2dcf49cebd;p=libs%2Fgl.git

diff --git a/source/glsl/validate.cpp b/source/glsl/validate.cpp
index 13cb5ccb..b8710509 100644
--- a/source/glsl/validate.cpp
+++ b/source/glsl/validate.cpp
@@ -239,10 +239,16 @@ void ReferenceValidator::visit(InterfaceBlockReference &iface)
 
 void ReferenceValidator::visit(FunctionCall &call)
 {
-	if(!call.declaration && !call.constructor)
+	if((!call.constructor && !call.declaration) || (call.constructor && !call.type))
 	{
-		map<string, FunctionDeclaration *>::iterator i = stage->functions.lower_bound(call.name);
-		if(i!=stage->functions.end() && i->second->name==call.name)
+		bool have_declaration = call.constructor;
+		if(!call.constructor)
+		{
+			map<string, FunctionDeclaration *>::iterator i = stage->functions.lower_bound(call.name);
+			have_declaration = (i!=stage->functions.end() && i->second->name==call.name);
+		}
+
+		if(have_declaration)
 		{
 			bool valid_types = true;
 			string signature;
@@ -255,7 +261,7 @@ void ReferenceValidator::visit(FunctionCall &call)
 			}
 
 			if(valid_types)
-				error(call, format("No matching overload found for call to '%s(%s)'", call.name, signature));
+				error(call, format("No matching %s found for '%s(%s)'", (call.constructor ? "constructor" : "overload"), call.name, signature));
 		}
 		else
 			error(call, format("Call to undeclared function '%s'", call.name));
@@ -338,8 +344,14 @@ void ExpressionValidator::visit(UnaryExpression &unary)
 void ExpressionValidator::visit(BinaryExpression &binary)
 {
 	if(!binary.type && binary.left->type && binary.right->type)
-		error(binary, format("No matching operator '%s' found for '%s' and '%s'",
-			binary.oper->token, binary.left->type->name, binary.right->type->name));
+	{
+		if(binary.oper->token[0]=='[')
+			error(binary, format("Can't index element of '%s' with '%s'",
+				binary.left->type->name, binary.right->type->name));
+		else
+			error(binary, format("No matching operator '%s' found for '%s' and '%s'",
+				binary.oper->token, binary.left->type->name, binary.right->type->name));
+	}
 	TraversingVisitor::visit(binary);
 }