X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Foutput.cpp;h=52371d1bfdd79f9f7de91ff457d10001952f6c62;hb=041ba4b1acd55337239c5ce24cc310118c621206;hp=3fc703c1966d9b4a9c745f7f3ee8e49aa55c5af8;hpb=c0be4e4ae1a8b6ac31ff6b7080e2242c13d947ff;p=libs%2Fgl.git

diff --git a/source/glsl/output.cpp b/source/glsl/output.cpp
index 3fc703c1..52371d1b 100644
--- a/source/glsl/output.cpp
+++ b/source/glsl/output.cpp
@@ -44,7 +44,7 @@ const string &Formatter::apply(Stage &s, Compiler::Mode m)
 		append("#extension ext_texture_array: require\n");
 	formatted += '\n';
 
-	visit(s.content);
+	s.content.visit(*this);
 
 	return formatted;
 }
@@ -82,6 +82,32 @@ void Formatter::set_source(unsigned index, unsigned line)
 	source_line = line;
 }
 
+void Formatter::visit(Block &block)
+{
+	unsigned brace_indent = indent;
+	bool use_braces = (block.use_braces || (indent && block.body.size()!=1));
+	if(use_braces)
+		append(format("%s{\n", string(brace_indent*2, ' ')));
+
+	SetForScope<unsigned> set(indent, indent+(indent>0 || use_braces));
+	string spaces(indent*2, ' ');
+	bool first = true;
+	for(NodeList<Statement>::iterator i=block.body.begin(); i!=block.body.end(); ++i)
+	{
+		if((*i)->source<=BUILTIN_SOURCE)
+			continue;
+		if(!first)
+			append('\n');
+		first = false;
+		set_source((*i)->source, (*i)->line);
+		append(spaces);
+		(*i)->visit(*this);
+	}
+
+	if(use_braces)
+		append(format("\n%s}", string(brace_indent*2, ' ')));
+}
+
 void Formatter::visit(Literal &literal)
 {
 	append(literal.token);
@@ -99,6 +125,11 @@ void Formatter::visit(VariableReference &var)
 	append(var.name);
 }
 
+void Formatter::visit(InterfaceBlockReference &iface)
+{
+	append(iface.name);
+}
+
 void Formatter::visit(MemberAccess &memacc)
 {
 	memacc.left->visit(*this);
@@ -107,25 +138,26 @@ void Formatter::visit(MemberAccess &memacc)
 
 void Formatter::visit(UnaryExpression &unary)
 {
-	if(unary.prefix)
-		append(unary.oper);
+	if(unary.oper->type==Operator::PREFIX)
+		append(unary.oper->token);
 	unary.expression->visit(*this);
-	if(!unary.prefix)
-		append(unary.oper);
+	if(unary.oper->type==Operator::POSTFIX)
+		append(unary.oper->token);
 }
 
 void Formatter::visit(BinaryExpression &binary)
 {
 	binary.left->visit(*this);
-	append(binary.oper);
+	append(binary.oper->token);
 	binary.right->visit(*this);
-	append(binary.after);
+	if(binary.oper->token[0]=='[')
+		append(']');
 }
 
 void Formatter::visit(Assignment &assign)
 {
 	assign.left->visit(*this);
-	append(format(" %s ", assign.oper));
+	append(format(" %s ", assign.oper->token));
 	assign.right->visit(*this);
 }
 
@@ -147,28 +179,6 @@ void Formatter::visit(ExpressionStatement &expr)
 	append(';');
 }
 
-void Formatter::visit(Block &block)
-{
-	unsigned brace_indent = indent;
-	bool use_braces = (block.use_braces || (indent && block.body.size()!=1));
-	if(use_braces)
-		append(format("%s{\n", string(brace_indent*2, ' ')));
-
-	SetForScope<unsigned> set(indent, indent+(indent>0 || use_braces));
-	string spaces(indent*2, ' ');
-	for(NodeList<Statement>::iterator i=block.body.begin(); i!=block.body.end(); ++i)
-	{
-		if(i!=block.body.begin())
-			append('\n');
-		set_source((*i)->source, (*i)->line);
-		append(spaces);
-		(*i)->visit(*this);
-	}
-
-	if(use_braces)
-		append(format("\n%s}", string(brace_indent*2, ' ')));
-}
-
 void Formatter::visit(Import &import)
 {
 	append(format("import %s;", import.module));
@@ -186,9 +196,9 @@ void Formatter::visit(Layout &layout)
 	{
 		if(i!=layout.qualifiers.begin())
 			append(", ");
-		append(i->identifier);
-		if(!i->value.empty())
-			append(format("=%s", i->value));
+		append(i->name);
+		if(i->has_value)
+			append(format("=%d", i->value));
 	}
 	append(')');
 }
@@ -219,7 +229,7 @@ void Formatter::visit(VariableDeclaration &var)
 		append(format("%s ", var.interpolation));
 	if(!var.sampling.empty())
 		append(format("%s ", var.sampling));
-	if(!var.interface.empty() && var.interface!=block_interface)
+	if(!var.interface.empty())
 	{
 		string interface = var.interface;
 		if(mode==Compiler::PROGRAM && stage && stage->required_features.glsl_version<Version(1, 30))
@@ -233,7 +243,10 @@ void Formatter::visit(VariableDeclaration &var)
 	}
 	if(!var.precision.empty())
 		append(format("%s ", var.precision));
-	append(format("%s %s", var.type, var.name));
+	string type_name = var.type_declaration->name;
+	if(var.array)
+		type_name = type_name.substr(0, type_name.find('['));
+	append(format("%s %s", type_name, var.name));
 	if(var.array)
 	{
 		append('[');
@@ -252,15 +265,22 @@ void Formatter::visit(VariableDeclaration &var)
 
 void Formatter::visit(InterfaceBlock &iface)
 {
-	SetForScope<string> set(block_interface, iface.interface);
 	append(format("%s %s\n", iface.interface, iface.name));
-	iface.members.visit(*this);
+	if(iface.struct_declaration)
+		iface.struct_declaration->members.visit(*this);
+	if(!iface.instance_name.empty())
+	{
+		append(' ');
+		append(iface.instance_name);
+		if(iface.array)
+			append("[]");
+	}
 	append(';');
 }
 
 void Formatter::visit(FunctionDeclaration &func)
 {
-	append(format("%s %s(", func.return_type, func.name));
+	append(format("%s %s(", func.return_type_declaration->name, func.name));
 	for(NodeArray<VariableDeclaration>::iterator i=func.parameters.begin(); i!=func.parameters.end(); ++i)
 	{
 		if(i!=func.parameters.begin())
@@ -341,6 +361,18 @@ void Formatter::visit(Iteration &iter)
 	}
 }
 
+void Formatter::visit(Passthrough &pass)
+{
+	append("passthrough");
+	if(pass.subscript)
+	{
+		append('[');
+		pass.subscript->visit(*this);
+		append(']');
+	}
+	append(';');
+}
+
 void Formatter::visit(Return &ret)
 {
 	append("return");