X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fglsl%2Foutput.cpp;h=6ba06447a3b02e6254b128884c72a206813ee22b;hb=c4aeeced7b397d46772577775bd3a0d6c4706cba;hp=c15c8c0421b9aae64add279cf11b9c05a03753f6;hpb=2280a09a286bbff2eb41783f8023863f320a3736;p=libs%2Fgl.git

diff --git a/source/glsl/output.cpp b/source/glsl/output.cpp
index c15c8c04..6ba06447 100644
--- a/source/glsl/output.cpp
+++ b/source/glsl/output.cpp
@@ -59,8 +59,8 @@ string Formatter::apply(Stage &s)
 void Formatter::append(const string &text)
 {
 	formatted += text;
-	for(string::const_iterator i=text.begin(); i!=text.end(); ++i)
-		if(*i=='\n')
+	for(char c: text)
+		if(c=='\n')
 			++source_line;
 }
 
@@ -99,16 +99,16 @@ void Formatter::visit(Block &block)
 	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)
+	for(const RefPtr<Statement> &s: block.body)
 	{
-		if(omit_builtin && (*i)->source<=BUILTIN_SOURCE)
+		if(omit_builtin && s->source<=BUILTIN_SOURCE)
 			continue;
 		if(!first)
 			append('\n');
 		first = false;
-		set_source((*i)->source, (*i)->line);
+		set_source(s->source, s->line);
 		append(spaces);
-		(*i)->visit(*this);
+		s->visit(*this);
 	}
 
 	if(use_braces)
@@ -171,6 +171,7 @@ void Formatter::visit(MemberAccess &memacc)
 	if(!r_empty_name)
 		append('.');
 	append(memacc.member);
+	r_empty_name = false;
 }
 
 void Formatter::visit(Swizzle &swizzle)
@@ -217,7 +218,7 @@ void Formatter::visit(TernaryExpression &ternary)
 void Formatter::visit(FunctionCall &call)
 {
 	append(format("%s(", call.name));
-	for(NodeArray<Expression>::iterator i=call.arguments.begin(); i!=call.arguments.end(); ++i)
+	for(auto i=call.arguments.begin(); i!=call.arguments.end(); ++i)
 	{
 		if(i!=call.arguments.begin())
 			append(", ");
@@ -245,7 +246,7 @@ void Formatter::visit(Precision &prec)
 void Formatter::visit(Layout &layout)
 {
 	append("layout(");
-	for(vector<Layout::Qualifier>::const_iterator i=layout.qualifiers.begin(); i!=layout.qualifiers.end(); ++i)
+	for(auto i=layout.qualifiers.begin(); i!=layout.qualifiers.end(); ++i)
 	{
 		if(i!=layout.qualifiers.begin())
 			append(", ");
@@ -339,7 +340,7 @@ void Formatter::visit(InterfaceBlock &iface)
 void Formatter::visit(FunctionDeclaration &func)
 {
 	append(format("%s %s(", func.return_type_declaration->name, func.name));
-	for(NodeArray<VariableDeclaration>::iterator i=func.parameters.begin(); i!=func.parameters.end(); ++i)
+	for(auto i=func.parameters.begin(); i!=func.parameters.end(); ++i)
 	{
 		if(i!=func.parameters.begin())
 			append(", ");