gl renderer: Don't copy colors into render ops

diff --git a/gsk/gl/gskglrenderer.c b/gsk/gl/gskglrenderer.c
index 137f64d3ea65be946ea671dd52d7f9da4e91bbfa..bcd221ac6bfb6281c7909372d475b6e9f71de2b7 100644 (file)
--- a/gsk/gl/gskglrenderer.c
+++ b/gsk/gl/gskglrenderer.c
@@ -23,6 +23,7 @@
 #include "gdk/gdkgltextureprivate.h"
 #include "gdk/gdkglcontextprivate.h"
 #include "gdk/gdkprofilerprivate.h"
+#include "gdk/gdkrgbaprivate.h"
 
 #include <epoxy/gl.h>
 #include <cairo-ft.h>
@@ -398,8 +399,10 @@ add_rect_outline_ops (GskGLRenderer         *self,
                       RenderOpBuilder       *builder,
                       const graphene_rect_t *rect)
 {
+  GdkRGBA *color = gdk_rgba_copy (&GDK_RGBA ("#F00")); /* Leaked */
+
   ops_set_program (builder, &self->color_program);
-  ops_set_color (builder, &(GdkRGBA) { 1, 0, 0, 1 });
+  ops_set_color (builder, color);
 
   add_rect_ops (builder,
                 &GRAPHENE_RECT_INIT (rect->origin.x, rect->origin.y,
@@ -2122,9 +2125,13 @@ static inline void
 apply_color_op (const Program  *program,
                 const RenderOp *op)
 {
-  OP_PRINT (" -> Color: (%f, %f, %f, %f)", op->color.red, op->color.green, op->color.blue, op->color.alpha);
+  OP_PRINT (" -> Color: (%f, %f, %f, %f)",
+            op->color->red, op->color->green, op->color->blue, op->color->alpha);
   glUniform4f (program->color.color_location,
-               op->color.red, op->color.green, op->color.blue, op->color.alpha);
+               op->color->red,
+               op->color->green,
+               op->color->blue,
+               op->color->alpha);
 }
 
 static inline void

diff --git a/gsk/gl/gskglrenderops.c b/gsk/gl/gskglrenderops.c
index f7105f202a5b2fb0c3ef00c966cd899169994d45..98736bcaca57ff64328d993e9be0516fe2bd2dec 100644 (file)
--- a/gsk/gl/gskglrenderops.c
+++ b/gsk/gl/gskglrenderops.c
@@ -649,7 +649,7 @@ ops_set_color (RenderOpBuilder *builder,
   current_program_state->color = *color;
 
   op = ops_begin (builder, OP_CHANGE_COLOR);
-  op->color = *color;
+  op->color = color;
 }
 
 void

diff --git a/gsk/gl/gskglrenderopsprivate.h b/gsk/gl/gskglrenderopsprivate.h
index 91ff2faa88b97129a077324e2bae8c451e9179be..5f43eace61c66f3fc3f9c9a016ae9ea75a59088f 100644 (file)
--- a/gsk/gl/gskglrenderopsprivate.h
+++ b/gsk/gl/gskglrenderopsprivate.h
@@ -159,7 +159,7 @@ typedef struct
     const Program *program;
     int texture_id;
     int render_target_id;
-    GdkRGBA color;
+    const GdkRGBA *color;
     GskQuadVertex vertex_data[6];
     GskRoundedRect clip;
     graphene_rect_t viewport;