X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=blobdiff_plain;f=source%2Fglsl%2Fresolve.cpp;h=406d7780e810181ffc8650e9c0ba8c20466d67aa;hp=eb99bec2ab57d9c801724b8803a3397e270ae363;hb=e1dacd64ec555d7344d32a9c11e785f1dc3d0929;hpb=b125c684e2c2feb86845f051285fe666d8ebfc76

diff --git a/source/glsl/resolve.cpp b/source/glsl/resolve.cpp
index eb99bec2..406d7780 100644
--- a/source/glsl/resolve.cpp
+++ b/source/glsl/resolve.cpp
@@ -972,6 +972,11 @@ void ExpressionResolver::visit_constructor(FunctionCall &call)
 		{
 			unsigned column_count = basic->size&0xFFFF;
 			unsigned row_count = basic->size>>16;
+
+			vector<RefPtr<Expression> > columns;
+			columns.reserve(column_count);
+			bool changed_columns = false;
+
 			if(call.arguments.size()==1)
 			{
 				/* A matrix can be constructed from a single element or another
@@ -994,8 +999,6 @@ void ExpressionResolver::visit_constructor(FunctionCall &call)
 					current_block->body.insert(insert_point, temporary);
 
 					// Create expressions to build each column.
-					vector<RefPtr<Expression> > columns;
-					columns.reserve(column_count);
 					for(unsigned j=0; j<column_count; ++j)
 					{
 						RefPtr<VariableReference> ref = new VariableReference;
@@ -1016,13 +1019,7 @@ void ExpressionResolver::visit_constructor(FunctionCall &call)
 							truncate_vector(columns.back(), row_count);
 					}
 
-					call.arguments.resize(column_count);
-					copy(columns.begin(), columns.end(), call.arguments.begin());
-
-					/* Let VariableResolver process the new nodes and finish
-					resolving the constructor on the next pass. */
-					r_any_resolved = true;
-					return;
+					changed_columns = true;
 				}
 				else
 					return;
@@ -1031,9 +1028,6 @@ void ExpressionResolver::visit_constructor(FunctionCall &call)
 			{
 				/* Construct a matrix from individual components in column-major
 				order.  Arguments must align at column boundaries. */
-				vector<RefPtr<Expression> > columns;
-				columns.reserve(column_count);
-
 				vector<RefPtr<Expression> > column_args;
 				column_args.reserve(row_count);
 				unsigned column_component_count = 0;
@@ -1067,11 +1061,24 @@ void ExpressionResolver::visit_constructor(FunctionCall &call)
 						else if(column_component_count>row_count)
 							// Argument alignment mismatch.
 							return;
+
+						changed_columns = true;
 					}
 				}
 			}
 			else
 				return;
+
+			if(changed_columns)
+			{
+				call.arguments.resize(column_count);
+				copy(columns.begin(), columns.end(), call.arguments.begin());
+
+				/* Let VariableResolver process the new nodes and finish
+				resolving the constructor on the next pass. */
+				r_any_resolved = true;
+				return;
+			}
 		}
 		else
 			return;