From: Kenneth Graunke <kenneth@whitecape.org>
Date: Fri, 9 Jul 2010 00:59:56 +0000 (-0700)
Subject: ast_function: Remove unnecessary check for empty constructors.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=59df3385e1c413332c75be5d0e7751972d45430e;p=mesa.git

ast_function: Remove unnecessary check for empty constructors.

This case is already caught by a later check that ensures sufficient
components were provided, based on the type.
---

diff --git a/src/glsl/ast_function.cpp b/src/glsl/ast_function.cpp
index b6811153879..c6cc3eb43c2 100644
--- a/src/glsl/ast_function.cpp
+++ b/src/glsl/ast_function.cpp
@@ -923,15 +923,6 @@ ast_function_expression::hir(exec_list *instructions,
 
 	 bool all_parameters_are_constant = true;
 
-	 /* This handles invalid constructor calls such as 'vec4 v = vec4();'
-	  */
-	 if (this->expressions.is_empty()) {
-	    _mesa_glsl_error(& loc, state, "too few components to construct "
-			     "`%s'",
-			     constructor_type->name);
-	    return ir_call::get_error_instruction(ctx);
-	 }
-
 	 foreach_list (n, &this->expressions) {
 	    ast_node *ast = exec_node_data(ast_node, n, link);
 	    ir_rvalue *result =