X-Git-Url: https://git.sesse.net/?p=movit;a=blobdiff_plain;f=effect_chain.h;h=642f079201f037219b1ead3c02aebf3e85bfb28d;hp=1078027a7bfc3b859531ddd6932ff79c96cc1795;hb=1f4a7aefd4a149e93a1dc41f85958e827670c98b;hpb=edb0700c0a8ea225ce9add1cb4f70d42af6de381

diff --git a/effect_chain.h b/effect_chain.h
index 1078027..642f079 100644
--- a/effect_chain.h
+++ b/effect_chain.h
@@ -26,6 +26,8 @@
 
 #include "image_format.h"
 
+namespace movit {
+
 class Effect;
 class Input;
 struct Phase;
@@ -57,9 +59,6 @@ public:
 	std::vector<Node *> incoming_links;
 
 private:
-	// Identifier used to create unique variables in GLSL.
-	std::string effect_id;
-
 	// Logical size of the output of this effect, ie. the resolution
 	// you would get if you sampled it as a texture. If it is undefined
 	// (since the inputs differ in resolution), it will be 0x0.
@@ -67,11 +66,9 @@ private:
 	// they will be equal.
 	unsigned output_width, output_height;
 
-	// If output goes to RTT (otherwise, none of these are set).
-	// The Phase pointer is a but ugly; we should probably fix so
-	// that Phase takes other phases as inputs, instead of Node.
-	GLuint output_texture;
-	unsigned output_texture_width, output_texture_height;
+	// If output goes to RTT, which phase it is in (otherwise unset).
+	// This is a bit ugly; we should probably fix so that Phase takes other
+	// phases as inputs, instead of Node.
 	Phase *phase;
 
 	// Used during the building of the effect chain.
@@ -93,6 +90,10 @@ struct Phase {
 
 	std::vector<Node *> effects;  // In order.
 	unsigned output_width, output_height, virtual_output_width, virtual_output_height;
+
+	// Identifier used to create unique variables in GLSL.
+	// Unique per-phase to increase cacheability of compiled shaders.
+	std::map<Node *, std::string> effect_ids;
 };
 
 class EffectChain {
@@ -173,6 +174,12 @@ public:
 	void replace_sender(Node *new_sender, Node *receiver);
 	void insert_node_between(Node *sender, Node *middle, Node *receiver);
 
+	// Get the current resource pool assigned to this EffectChain.
+	// Primarily to let effects allocate textures as needed.
+	// Any resources you get from the pool must be returned to the pool
+	// no later than in the Effect's destructor.
+	ResourcePool *get_resource_pool() { return resource_pool; }
+
 private:
 	// Make sure the output rectangle is at least large enough to hold
 	// the given input rectangle in both dimensions, and is of the
@@ -261,4 +268,6 @@ private:
 	bool owns_resource_pool;
 };
 
+}  // namespace movit
+
 #endif // !defined(_MOVIT_EFFECT_CHAIN_H)