X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;ds=sidebyside;f=source%2Fglsl%2Foptimize.h;h=01508f450257ac4bee4b00dffd4e062449f3e8a5;hb=9a63244c1342337915c4610401a24c09fa72cc3d;hp=698aa0b2617efd3ef1f6a98348aa754b19f04862;hpb=696a97bd7411d69953c1a9e4b5f3dfb4c1d848f1;p=libs%2Fgl.git

diff --git a/source/glsl/optimize.h b/source/glsl/optimize.h
index 698aa0b2..01508f45 100644
--- a/source/glsl/optimize.h
+++ b/source/glsl/optimize.h
@@ -10,11 +10,8 @@ namespace Msp {
 namespace GL {
 namespace SL {
 
-class InlineableFunctionLocator: public StageVisitor
+class InlineableFunctionLocator: private TraversingVisitor
 {
-public:
-	typedef std::set<FunctionDeclaration *> ResultType;
-
 private:
 	std::map<FunctionDeclaration *, unsigned> refcounts;
 	std::set<FunctionDeclaration *> inlineable;
@@ -23,13 +20,15 @@ private:
 public:
 	InlineableFunctionLocator();
 
-	const ResultType &get_result() const { return inlineable; }
-	using StageVisitor::visit;
+	const std::set<FunctionDeclaration *> &apply(Stage &s) { visit(s.content); return inlineable; }
+
+private:
 	virtual void visit(FunctionCall &);
 	virtual void visit(FunctionDeclaration &);
+	using TraversingVisitor::visit;
 };
 
-class FunctionInliner: public StageVisitor
+class FunctionInliner: private TraversingVisitor
 {
 private:
 	std::set<FunctionDeclaration *> inlineable;
@@ -37,13 +36,13 @@ private:
 	RefPtr<Expression> inline_result;
 
 public:
-	FunctionInliner();
 	FunctionInliner(const std::set<FunctionDeclaration *> &);
 
+	void apply(Stage &s) { visit(s.content); }
+
 private:
 	void visit_and_inline(RefPtr<Expression> &);
-public:
-	using StageVisitor::visit;
+
 	virtual void visit(Block &);
 	virtual void visit(UnaryExpression &);
 	virtual void visit(BinaryExpression &);
@@ -51,9 +50,10 @@ public:
 	virtual void visit(FunctionCall &);
 	virtual void visit(VariableDeclaration &);
 	virtual void visit(Return &);
+	using TraversingVisitor::visit;
 };
 
-class ConstantConditionEliminator: public BlockModifier
+class ConstantConditionEliminator: private BlockModifier
 {
 private:
 	unsigned scope_level;
@@ -63,20 +63,20 @@ private:
 public:
 	ConstantConditionEliminator();
 
-	using StageVisitor::visit;
+	void apply(Stage &s) { visit(s.content); }
+
+private:
 	virtual void visit(Block &);
 	virtual void visit(UnaryExpression &);
 	virtual void visit(Assignment &);
 	virtual void visit(VariableDeclaration &);
 	virtual void visit(Conditional &);
 	virtual void visit(Iteration &);
+	using BlockModifier::visit;
 };
 
-class UnusedVariableLocator: public StageVisitor
+class UnusedVariableLocator: private TraversingVisitor
 {
-public:
-	typedef std::set<Node *> ResultType;
-
 private:
 	struct VariableInfo
 	{
@@ -102,10 +102,9 @@ private:
 public:
 	UnusedVariableLocator();
 
-	virtual void apply(Stage &);
-	const ResultType &get_result() const { return unused_nodes; }
+	const std::set<Node *> &apply(Stage &);
+
 private:
-	using StageVisitor::visit;
 	virtual void visit(VariableReference &);
 	virtual void visit(MemberAccess &);
 	virtual void visit(BinaryExpression &);
@@ -120,22 +119,22 @@ private:
 	void merge_down_variables();
 	virtual void visit(Conditional &);
 	virtual void visit(Iteration &);
+	using TraversingVisitor::visit;
 };
 
-class UnusedFunctionLocator: public StageVisitor
+class UnusedFunctionLocator: private TraversingVisitor
 {
-public:
-	typedef std::set<Node *> ResultType;
-
 private:
 	std::set<Node *> unused_nodes;
 	std::set<FunctionDeclaration *> used_definitions;
 
 public:
-	const ResultType &get_result() const { return unused_nodes; }
-	using StageVisitor::visit;
+	const std::set<Node *> &apply(Stage &s) { visit(s.content); return unused_nodes; }
+
+private:
 	virtual void visit(FunctionCall &);
 	virtual void visit(FunctionDeclaration &);
+	using TraversingVisitor::visit;
 };
 
 } // namespace SL