Fix runtime error
[geeqie.git] / src / renderer-clutter.c
index a67b950..3937275 100644 (file)
@@ -1,14 +1,22 @@
 /*
- * Geeqie
- * (C) 2006 John Ellis
- * Copyright (C) 2008 - 2012 The Geeqie Team
+ * Copyright (C) 2006 John Ellis
+ * Copyright (C) 2008 - 2016 The Geeqie Team
  *
  * Author: John Ellis
- * Author: Vladimir Nadvornik
  *
- * This software is released under the GNU General Public License (GNU GPL).
- * Please read the included file COPYING for more information.
- * This software comes with no warranty of any kind, use at your own risk!
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
 #include <stdio.h>
 
 #ifdef HAVE_CLUTTER
 
+/* for 3d texture */
+#define COGL_ENABLE_EXPERIMENTAL_API
+#define CLUTTER_ENABLE_EXPERIMENTAL_API
+
 #include <clutter/clutter.h>
 
 #include <clutter-gtk/clutter-gtk.h>
@@ -85,21 +97,26 @@ struct _RendererClutter
        RendererFuncs f;
        PixbufRenderer *pr;
 
-       
+
        gint stereo_mode;
        gint stereo_off_x;
        gint stereo_off_y;
-       
-       
+
+
        GList *pending_updates;
        gint idle_update;
-       
+
        GList *overlay_list;
-       
+
        GtkWidget *widget; /* widget and stage may be shared with other renderers */
        ClutterActor *stage;
        ClutterActor *texture;
        ClutterActor *group;
+
+       gboolean clut_updated;
+       gint64 last_pixbuf_change;
+
+       GdkPixbuf *display_pixbuf;
 };
 
 typedef struct _RendererClutterAreaParam RendererClutterAreaParam;
@@ -111,209 +128,149 @@ struct _RendererClutterAreaParam {
        gint h;
 };
 
-static void rc_set_shader(CoglHandle material)
-{
-  CoglHandle shader;
-  CoglHandle program;
-
-  shader = cogl_create_shader (COGL_SHADER_TYPE_FRAGMENT);
-  cogl_shader_source (shader,
-  "vec3 checker(vec2 texc, vec3 color0, vec3 color1)"
-  "{"
-  "  if (mod(int(floor(texc.x) + floor(texc.y)), 2) == 0)"
-  "    return color0;"
-  "  else"
-  "    return color1;"
-  "}"
-  
-  "uniform sampler2D tex;"
-  "void main(void)"
-  "{"
-  "    vec3 bg = checker(gl_FragCoord.xy / 16, vec3(0.6, 0.6, 0.6), vec3(0.4, 0.4, 0.4));"
-  "    vec4 img4 = texture2D(tex, gl_TexCoord[0].xy);"
-  "    vec3 img3 = img4.rgb;"
-  "    gl_FragColor = vec4(img3 * img4.a + bg * (1.0 - img4.a), 1.0);"
-  "}"
-  );
-  cogl_shader_compile(shader);
-  gchar *err = cogl_shader_get_info_log(shader);
-  printf("%s\n",err);
-  g_free(err);
-
-  program = cogl_create_program ();
-  cogl_program_attach_shader (program, shader);
-  cogl_handle_unref (shader);
-  cogl_program_link (program);
-
-  gint uniform_no = cogl_program_get_uniform_location (program, "tex");
-  cogl_program_set_uniform_1i (program, uniform_no, 0);
-
-  cogl_material_set_user_program (material, program);
-  cogl_handle_unref (program);
-}
-
-
 static void rc_sync_actor(RendererClutter *rc)
 {
        PixbufRenderer *pr = rc->pr;
        gint anchor_x = 0;
        gint anchor_y = 0;
-       
+       gint rot_z = 0;
+       gint rot_y = 0;
+
        clutter_actor_set_anchor_point(CLUTTER_ACTOR(rc->texture), 0, 0);
 
-       printf("scale %d %d\n", rc->pr->width, rc->pr->height);
-       printf("pos   %d %d\n", rc->pr->x_offset, rc->pr->y_offset);
-       
-       clutter_actor_set_scale(CLUTTER_ACTOR(rc->texture), 
+       DEBUG_3("scale %d %d", rc->pr->width, rc->pr->height);
+       DEBUG_3("pos   %d %d", rc->pr->x_offset, rc->pr->y_offset);
+
+       clutter_actor_set_scale(CLUTTER_ACTOR(rc->texture),
                                (gfloat)pr->width / pr->image_width,
                                (gfloat)pr->height / pr->image_height);
-                               
+
        switch (pr->orientation)
                {
                case EXIF_ORIENTATION_TOP_LEFT:
-                       /* normal  */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               0, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               0, 0, 0, 0);
-                       anchor_x = 0;
-                       anchor_y = 0;
+                       /* 1 - Horizontal (normal)  */
                        break;
                case EXIF_ORIENTATION_TOP_RIGHT:
-                       /* mirrored */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               0, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               180, 0, 0, 0);
+                       /* 2 - Mirror horizontal */
+                       rot_y = 180;
                        anchor_x = pr->width;
-                       anchor_y = 0;
                        break;
                case EXIF_ORIENTATION_BOTTOM_RIGHT:
-                       /* upside down */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               180, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               0, 0, 0, 0);
+                       /* 3 - Rotate 180 */
+                       rot_z = 180;
                        anchor_x = pr->width;
                        anchor_y = pr->height;
                        break;
                case EXIF_ORIENTATION_BOTTOM_LEFT:
-                       /* flipped */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               180, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               180, 0, 0, 0);
-                       anchor_x = 0;
+                       /* 4 - Mirror vertical */
+                       rot_z = 180;
+                       rot_y = 180;
                        anchor_y = pr->height;
                        break;
                case EXIF_ORIENTATION_LEFT_TOP:
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               -90, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               180, 0, 0, 0);
-                       anchor_x = 0;
-                       anchor_y = 0;
+                       /* 5 - Mirror horizontal and rotate 270 CW */
+                       rot_z = 270;
+                       rot_y = 180;
                        break;
                case EXIF_ORIENTATION_RIGHT_TOP:
-                       /* rotated -90 (270) */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               -90, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               0, 0, 0, 0);
-                       anchor_x = 0;
-                       anchor_y = pr->height;
+                       /* 6 - Rotate 90 CW */
+                       rot_z = 90;
+                       anchor_x = pr->width;
                        break;
                case EXIF_ORIENTATION_RIGHT_BOTTOM:
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               90, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               180, 0, 0, 0);
+                       /* 7 - Mirror horizontal and rotate 90 CW */
+                       rot_z = 90;
+                       rot_y = 180;
                        anchor_x = pr->width;
                        anchor_y = pr->height;
                        break;
                case EXIF_ORIENTATION_LEFT_BOTTOM:
-                       /* rotated 90 */
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Z_AXIS,
-                                               90, 0, 0, 0);
-                       clutter_actor_set_rotation(CLUTTER_ACTOR(rc->texture),
-                                               CLUTTER_Y_AXIS,
-                                               0, 0, 0, 0);
-                       anchor_x = pr->width;
-                       anchor_y = 0;
+                       /* 8 - Rotate 270 CW */
+                       rot_z = 270;
+                       anchor_y = pr->height;
                        break;
                default:
                        /* The other values are out of range */
                        break;
                }
-       
-       clutter_actor_set_position(CLUTTER_ACTOR(rc->texture), 
-                               pr->x_offset - pr->x_scroll + anchor_x, 
+
+       clutter_actor_set_rotation(     CLUTTER_ACTOR(rc->texture),
+                                                               CLUTTER_Z_AXIS,
+                                                               rot_z, 0, 0, 0);
+       clutter_actor_set_rotation(     CLUTTER_ACTOR(rc->texture),
+                                                               CLUTTER_Y_AXIS,
+                                                               rot_y, 0, 0, 0);
+
+       clutter_actor_set_position(CLUTTER_ACTOR(rc->texture),
+                               pr->x_offset - pr->x_scroll + anchor_x,
                                pr->y_offset - pr->y_scroll + anchor_y);
 
 }
 
 
-static void renderer_area_clip_add(RendererClutter *rc, gfloat x, gfloat y, gfloat w, gfloat h)
+static void rc_area_clip_add(RendererClutter *rc, gfloat x, gfloat y, gfloat w, gfloat h)
 {
-       PixbufRenderer *pr = rc->pr;
        gfloat x2, y2;
        gfloat clip_x, clip_y, clip_w, clip_h, clip_x2, clip_y2;
-       
+
        x2 = x + w;
        y2 = y + h;
-       
+
        clutter_actor_get_clip(rc->texture, &clip_x, &clip_y, &clip_w, &clip_h);
-       
+
        clip_x2 = clip_x + clip_w;
        clip_y2 = clip_y + clip_h;
-       
+
        if (clip_x > x) clip_x = x;
        if (clip_x2 < x2) clip_x2 = x2;
        if (clip_y > y) clip_y = y;
        if (clip_y2 < y2) clip_y2 = y2;
-       
+
        clip_w = clip_x2 - clip_x;
        clip_h = clip_y2 - clip_y;
-       
-       printf("clip %f %f %f %f\n", clip_x, clip_y, clip_w, clip_h);
+
        clutter_actor_set_clip(rc->texture, clip_x, clip_y, clip_w, clip_h);
 }
 
 #define MAX_REGION_AREA (32768 * 1024)
 
-static gboolean renderer_area_changed_cb(gpointer data)
+static gboolean rc_area_changed_cb(gpointer data);
+
+static void rc_schedule_texture_upload(RendererClutter *rc)
+{
+       if (g_get_monotonic_time() - rc->last_pixbuf_change < 50000)
+               {
+               /* delay clutter redraw until the texture has some data
+                  set priority between gtk redraw and clutter redraw */
+               DEBUG_3("%s tex upload high prio", get_exec_time());
+               rc->idle_update = g_idle_add_full(CLUTTER_PRIORITY_REDRAW - 10, rc_area_changed_cb, rc, NULL);
+               }
+       else
+               {
+               /* higher prio than histogram */
+               DEBUG_3("%s tex upload low prio", get_exec_time());
+
+               rc->idle_update = g_idle_add_full(G_PRIORITY_DEFAULT_IDLE - 5, rc_area_changed_cb, rc, NULL);
+               }
+}
+
+static void update_display_pixbuf(RendererClutter *rc);
+
+static gboolean rc_area_changed_cb(gpointer data)
 {
        RendererClutter *rc = (RendererClutter *)data;
        PixbufRenderer *pr = rc->pr;
-       
+
        RendererClutterAreaParam *par = rc->pending_updates->data;
-       
+
        gint h = MAX_REGION_AREA / par->w;
        if (h == 0) h = 1;
        if (h > par->h) h = par->h;
-       
-       
-       printf("renderer_area_changed_cb %d %d %d %d  (%d)\n", par->x, par->y, par->w, h, par->h);
-       DEBUG_0("%s upload start", get_exec_time());
+
        if (pr->pixbuf)
                {
                CoglHandle texture = clutter_texture_get_cogl_texture(CLUTTER_TEXTURE(rc->texture));
-               
+
                cogl_texture_set_region(texture,
                                        par->x + GET_RIGHT_PIXBUF_OFFSET(rc),
                                        par->y,
@@ -327,13 +284,12 @@ static gboolean renderer_area_changed_cb(gpointer data)
                                        gdk_pixbuf_get_rowstride(pr->pixbuf),
                                        gdk_pixbuf_get_pixels(pr->pixbuf));
                }
-       DEBUG_0("%s upload end", get_exec_time());
-       renderer_area_clip_add(rc, par->x, par->y, par->w, h);
+       rc_area_clip_add(rc, par->x, par->y, par->w, h);
+
 
-               
        par->y += h;
        par->h -= h;
-       
+
        if (par->h == 0)
                {
                rc->pending_updates = g_list_remove(rc->pending_updates, par);
@@ -343,13 +299,43 @@ static gboolean renderer_area_changed_cb(gpointer data)
                {
                clutter_actor_queue_redraw(CLUTTER_ACTOR(rc->texture));
                rc->idle_update = 0;
+
+               update_display_pixbuf(rc);
+
+               if (pr->func_post_process && pr->pixbuf)
+                       {
+                       pr->func_post_process(pr, &rc->display_pixbuf, 0, 0, gdk_pixbuf_get_width(pr->pixbuf), gdk_pixbuf_get_height(pr->pixbuf), pr->post_process_user_data);
+                       }
+
+               if (pr->pixbuf)
+                       {
+                       CoglHandle texture = clutter_texture_get_cogl_texture(CLUTTER_TEXTURE(rc->texture));
+
+                       cogl_texture_set_region(texture,
+                                               0 + GET_RIGHT_PIXBUF_OFFSET(rc),
+                                               0,
+                                               0,
+                                               0,
+                                               gdk_pixbuf_get_width(pr->pixbuf),
+                                               gdk_pixbuf_get_height(pr->pixbuf),
+                                               gdk_pixbuf_get_width(pr->pixbuf),
+                                               gdk_pixbuf_get_height(pr->pixbuf),
+                                               gdk_pixbuf_get_has_alpha(pr->pixbuf) ? COGL_PIXEL_FORMAT_RGBA_8888 : COGL_PIXEL_FORMAT_RGB_888,
+                                               gdk_pixbuf_get_rowstride(pr->pixbuf),
+                                               gdk_pixbuf_get_pixels(rc->display_pixbuf));
+                       DEBUG_3("%s upload end", get_exec_time());
+                       }
+
                return FALSE;
                }
-       return TRUE;
+
+       rc_schedule_texture_upload(rc);
+
+       return FALSE; /* it was rescheduled, possibly with different prio */
 }
 
 
-static void renderer_area_changed(void *renderer, gint src_x, gint src_y, gint src_w, gint src_h)
+static void rc_area_changed(void *renderer, gint src_x, gint src_y, gint src_w, gint src_h)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
        PixbufRenderer *pr = rc->pr;
@@ -357,16 +343,16 @@ static void renderer_area_changed(void *renderer, gint src_x, gint src_y, gint s
 
        gint width = gdk_pixbuf_get_width(pr->pixbuf);
        gint height = gdk_pixbuf_get_height(pr->pixbuf);
-               
-       if (pr->stereo_data == STEREO_PIXBUF_SBS || pr->stereo_data == STEREO_PIXBUF_CROSS) 
+
+       if (pr->stereo_data == STEREO_PIXBUF_SBS || pr->stereo_data == STEREO_PIXBUF_CROSS)
                        {
                        width /= 2;
                        }
-       
+
        if (!pr_clip_region(src_x, src_y, src_w, src_h,
-                            GET_RIGHT_PIXBUF_OFFSET(rc), 0, width, height,
-                            &src_x, &src_y, &src_w, &src_h)) return;
-       
+                                               GET_RIGHT_PIXBUF_OFFSET(rc), 0, width, height,
+                                               &src_x, &src_y, &src_w, &src_h)) return;
+
        par = g_new0(RendererClutterAreaParam, 1);
        par->rc = rc;
        par->x = src_x - GET_RIGHT_PIXBUF_OFFSET(rc);
@@ -374,13 +360,13 @@ static void renderer_area_changed(void *renderer, gint src_x, gint src_y, gint s
        par->w = src_w;
        par->h = src_h;
        rc->pending_updates = g_list_append(rc->pending_updates, par);
-       if (!rc->idle_update) 
+       if (!rc->idle_update)
                {
-               rc->idle_update = g_idle_add_full(G_PRIORITY_DEFAULT_IDLE, renderer_area_changed_cb, rc, NULL);
+               rc_schedule_texture_upload(rc);
                }
 }
 
-static void renderer_remove_pending_updates(RendererClutter *rc)
+static void rc_remove_pending_updates(RendererClutter *rc)
 {
        if (rc->idle_update) g_idle_remove_by_data(rc);
        rc->idle_update = 0;
@@ -392,68 +378,127 @@ static void renderer_remove_pending_updates(RendererClutter *rc)
                }
 }
 
-static void renderer_update_pixbuf(void *renderer, gboolean lazy)
+static void update_display_pixbuf(RendererClutter *rc)
 {
-       RendererClutter *rc = (RendererClutter *)renderer;
        PixbufRenderer *pr = rc->pr;
-       
-       renderer_remove_pending_updates(rc);
-       
+       GdkPixbuf* tmppixbuf = NULL;
+
        if (pr->pixbuf)
                {
-               gint width = gdk_pixbuf_get_width(pr->pixbuf);
-               gint height = gdk_pixbuf_get_height(pr->pixbuf);
-               
-               gint prev_width, prev_height;
-               
-               if (pr->stereo_data == STEREO_PIXBUF_SBS || pr->stereo_data == STEREO_PIXBUF_CROSS) 
+               if (rc->display_pixbuf)
                        {
-                       width /= 2;
+                       g_object_unref(rc->display_pixbuf);
                        }
 
-               
-               printf("renderer_update_pixbuf\n");
-               clutter_texture_get_base_size(CLUTTER_TEXTURE(rc->texture), &prev_width, &prev_height);
-               printf("change from %d %d to %d %d\n", prev_width, prev_height, width, height);
-               
-               if (width != prev_width || height != prev_height)
+               if (!gdk_pixbuf_get_has_alpha(pr->pixbuf))
                        {
-                       /* FIXME use CoglMaterial with multiple textures for background, color management, anaglyph, ... */
-                       CoglHandle texture = cogl_texture_new_with_size(width,
-                                                                       height,
-                                                                       COGL_TEXTURE_NO_AUTO_MIPMAP,
-                                                                       gdk_pixbuf_get_has_alpha(pr->pixbuf) ? COGL_PIXEL_FORMAT_RGBA_8888 : COGL_PIXEL_FORMAT_RGB_888);
-
-                       if (texture != COGL_INVALID_HANDLE)
-                               {
-                               clutter_texture_set_cogl_texture(CLUTTER_TEXTURE(rc->texture), texture);
-                               cogl_handle_unref(texture);
-                               }
+                       rc->display_pixbuf = gdk_pixbuf_copy(pr->pixbuf);
                        }
-               clutter_actor_set_clip(rc->texture, 0, 0, 0, 0); /* visible area is extended as area_changed events arrive */
-               if (!lazy)
+               else
                        {
-                       renderer_area_changed(renderer, GET_RIGHT_PIXBUF_OFFSET(rc), 0, width, height);
+                       if (pr->ignore_alpha)
+                               {
+                               tmppixbuf = gdk_pixbuf_add_alpha(pr->pixbuf, FALSE, 0, 0, 0);
+
+                               pixbuf_ignore_alpha_rect(tmppixbuf, 0, 0, gdk_pixbuf_get_width(pr->pixbuf), gdk_pixbuf_get_height(pr->pixbuf));
+
+                               rc->display_pixbuf = gdk_pixbuf_composite_color_simple (tmppixbuf,
+                                               gdk_pixbuf_get_width(pr->pixbuf),
+                                               gdk_pixbuf_get_height(pr->pixbuf),
+                                               GDK_INTERP_BILINEAR,
+                                               255,
+                                               PR_ALPHA_CHECK_SIZE,
+                                               ((options->image.alpha_color_1.red << 8 & 0x00FF0000) +
+                                               (options->image.alpha_color_1.green & 0x00FF00) +
+                                               (options->image.alpha_color_1.blue >> 8 & 0x00FF)),
+                                               ((options->image.alpha_color_2.red << 8 & 0x00FF0000) +
+                                               (options->image.alpha_color_2.green & 0x00FF00) +
+                                               (options->image.alpha_color_2.blue >> 8 & 0x00FF)));
+                               g_object_unref(tmppixbuf);
+                               }
+                       else
+                               {
+                               rc->display_pixbuf = gdk_pixbuf_composite_color_simple (pr->pixbuf,
+                                               gdk_pixbuf_get_width(pr->pixbuf),
+                                               gdk_pixbuf_get_height(pr->pixbuf),
+                                               GDK_INTERP_BILINEAR,
+                                               255,
+                                               PR_ALPHA_CHECK_SIZE,
+                                               ((options->image.alpha_color_1.red << 8 & 0x00FF0000) +
+                                               (options->image.alpha_color_1.green & 0x00FF00) +
+                                               (options->image.alpha_color_1.blue >> 8 & 0x00FF)),
+                                               ((options->image.alpha_color_2.red << 8 & 0x00FF0000) +
+                                               (options->image.alpha_color_2.green & 0x00FF00) +
+                                               (options->image.alpha_color_2.blue >> 8 & 0x00FF)));
+                               }
                        }
                }
+}
 
+static void rc_update_pixbuf(void *renderer, gboolean lazy)
+{
+       RendererClutter *rc = (RendererClutter *)renderer;
+       PixbufRenderer *pr = rc->pr;
 
-       printf("renderer_update_pixbuf\n");
-       rc_sync_actor(rc);
+       DEBUG_3("rc_update_pixbuf");
+
+       rc_remove_pending_updates(rc);
+
+       rc->last_pixbuf_change = g_get_monotonic_time();
+       DEBUG_3("%s change time reset", get_exec_time());
+
+       if (pr->pixbuf)
+               {
+                       gint width = gdk_pixbuf_get_width(pr->pixbuf);
+                       gint height = gdk_pixbuf_get_height(pr->pixbuf);
+
+                       DEBUG_3("pixbuf size %d x %d (%d)", width, height, gdk_pixbuf_get_has_alpha(pr->pixbuf) ? 32 : 24);
+
+                       gint prev_width, prev_height;
+
+                       if (pr->stereo_data == STEREO_PIXBUF_SBS || pr->stereo_data == STEREO_PIXBUF_CROSS)
+                               {
+                               width /= 2;
+                               }
+
+
+                       clutter_texture_get_base_size(CLUTTER_TEXTURE(rc->texture), &prev_width, &prev_height);
+
+                       if (width != prev_width || height != prev_height)
+                               {
+                               /** @FIXME use CoglMaterial with multiple textures for background, color management, anaglyph, ... */
+                               CoglHandle texture = cogl_texture_new_with_size(width,
+                                                                               height,
+                                                                               COGL_TEXTURE_NO_AUTO_MIPMAP,
+                                                                               gdk_pixbuf_get_has_alpha(pr->pixbuf) ? COGL_PIXEL_FORMAT_RGBA_8888 : COGL_PIXEL_FORMAT_RGB_888);
+
+                               if (texture != COGL_INVALID_HANDLE)
+                                       {
+                                       clutter_texture_set_cogl_texture(CLUTTER_TEXTURE(rc->texture), texture);
+                                       cogl_handle_unref(texture);
+                                       }
+                               }
+                       clutter_actor_set_clip(rc->texture, 0, 0, 0, 0); /* visible area is extended as area_changed events arrive */
+                       if (!lazy)
+                               {
+                               rc_area_changed(renderer, GET_RIGHT_PIXBUF_OFFSET(rc), 0, width, height);
+                               }
+
+               rc->clut_updated = FALSE;
+               }
 }
 
 
 
-static void renderer_update_zoom(void *renderer, gboolean lazy)
+static void rc_update_zoom(void *renderer, gboolean lazy)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
-       PixbufRenderer *pr = rc->pr;
 
-       printf("renderer_update_zoom\n");
+       DEBUG_3("rc_update_zoom");
        rc_sync_actor(rc);
 }
 
-static void renderer_invalidate_region(void *renderer, gint x, gint y, gint w, gint h)
+static void rc_invalidate_region(void *renderer, gint x, gint y, gint w, gint h)
 {
 }
 
@@ -533,12 +578,7 @@ static void rc_overlay_free_all(RendererClutter *rc)
                }
 }
 
-
-static void renderer_overlay_draw(void *renderer, gint x, gint y, gint w, gint h)
-{
-}
-
-static gint renderer_overlay_add(void *renderer, GdkPixbuf *pixbuf, gint x, gint y, OverlayRendererFlags flags)
+static gint rc_overlay_add(void *renderer, GdkPixbuf *pixbuf, gint x, gint y, OverlayRendererFlags flags)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
        PixbufRenderer *pr = rc->pr;
@@ -558,12 +598,12 @@ static gint renderer_overlay_add(void *renderer, GdkPixbuf *pixbuf, gint x, gint
        od->x = x;
        od->y = y;
        od->flags = flags;
-       
+
        od->actor = gtk_clutter_texture_new();
        g_signal_connect (od->actor, "destroy", G_CALLBACK(rc_overlay_actor_destroy_cb), od);
-       
+
        gtk_clutter_texture_set_from_pixbuf(GTK_CLUTTER_TEXTURE (od->actor), pixbuf, NULL);
-       clutter_container_add_actor(CLUTTER_CONTAINER(rc->group), od->actor);
+       clutter_container_add_actor(CLUTTER_CONTAINER(rc->group), od->actor);
 
        rc->overlay_list = g_list_append(rc->overlay_list, od);
        rc_overlay_update_position(rc, od);
@@ -571,7 +611,7 @@ static gint renderer_overlay_add(void *renderer, GdkPixbuf *pixbuf, gint x, gint
        return od->id;
 }
 
-static void renderer_overlay_set(void *renderer, gint id, GdkPixbuf *pixbuf, gint x, gint y)
+static void rc_overlay_set(void *renderer, gint id, GdkPixbuf *pixbuf, gint x, gint y)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
        PixbufRenderer *pr = rc->pr;
@@ -600,7 +640,7 @@ static void renderer_overlay_set(void *renderer, gint id, GdkPixbuf *pixbuf, gin
                }
 }
 
-static gboolean renderer_overlay_get(void *renderer, gint id, GdkPixbuf **pixbuf, gint *x, gint *y)
+static gboolean rc_overlay_get(void *renderer, gint id, GdkPixbuf **pixbuf, gint *x, gint *y)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
 
@@ -620,25 +660,25 @@ static gboolean renderer_overlay_get(void *renderer, gint id, GdkPixbuf **pixbuf
 }
 
 
-static void renderer_update_sizes(void *renderer)
+static void rc_update_viewport(void *renderer)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
-       ClutterColor stage_color = { 0x00, 0x00, 0x00, 0xff }; 
+       ClutterColor stage_color = { rc->pr->color.red / 256, rc->pr->color.green / 256, rc->pr->color.blue / 256, 0xff };
 
        rc->stereo_off_x = 0;
        rc->stereo_off_y = 0;
-       
-       if (rc->stereo_mode & PR_STEREO_RIGHT) 
+
+       if (rc->stereo_mode & PR_STEREO_RIGHT)
                {
-               if (rc->stereo_mode & PR_STEREO_HORIZ) 
+               if (rc->stereo_mode & PR_STEREO_HORIZ)
                        {
                        rc->stereo_off_x = rc->pr->viewport_width;
                        }
-               else if (rc->stereo_mode & PR_STEREO_VERT) 
+               else if (rc->stereo_mode & PR_STEREO_VERT)
                        {
                        rc->stereo_off_y = rc->pr->viewport_height;
                        }
-               else if (rc->stereo_mode & PR_STEREO_FIXED) 
+               else if (rc->stereo_mode & PR_STEREO_FIXED)
                        {
                        rc->stereo_off_x = rc->pr->stereo_fixed_x_right;
                        rc->stereo_off_y = rc->pr->stereo_fixed_y_right;
@@ -646,25 +686,23 @@ static void renderer_update_sizes(void *renderer)
                }
        else
                {
-               if (rc->stereo_mode & PR_STEREO_FIXED) 
+               if (rc->stereo_mode & PR_STEREO_FIXED)
                        {
                        rc->stereo_off_x = rc->pr->stereo_fixed_x_left;
                        rc->stereo_off_y = rc->pr->stereo_fixed_y_left;
                        }
                }
-        DEBUG_1("update size: %p  %d %d   %d %d", rc, rc->stereo_off_x, rc->stereo_off_y, rc->pr->viewport_width, rc->pr->viewport_height);
+       DEBUG_3("rc_update_viewport  scale %d %d", rc->pr->width, rc->pr->height);
 
-       printf("renderer_update_sizes  scale %d %d\n", rc->pr->width, rc->pr->height);
-
-        clutter_stage_set_color(CLUTTER_STAGE(rc->stage), &stage_color);
+       clutter_stage_set_color(CLUTTER_STAGE(rc->stage), &stage_color);
 
 
        clutter_actor_set_size(rc->group, rc->pr->viewport_width, rc->pr->viewport_height);
        clutter_actor_set_position(rc->group, rc->stereo_off_x, rc->stereo_off_y);
-       
+
        clutter_actor_set_rotation(CLUTTER_ACTOR(rc->group),
                                                CLUTTER_Y_AXIS,
-                                               (rc->stereo_mode & PR_STEREO_MIRROR) ? 180 : 0, 
+                                               (rc->stereo_mode & PR_STEREO_MIRROR) ? 180 : 0,
                                                rc->pr->viewport_width / 2.0, 0, 0);
 
        clutter_actor_set_rotation(CLUTTER_ACTOR(rc->group),
@@ -676,44 +714,48 @@ static void renderer_update_sizes(void *renderer)
        rc_overlay_update_positions(rc);
 }
 
-static void renderer_scroll(void *renderer, gint x_off, gint y_off)
+static void rc_scroll(void *renderer, gint x_off, gint y_off)
 {
-       printf("renderer_scroll\n");
+       DEBUG_3("rc_scroll");
        RendererClutter *rc = (RendererClutter *)renderer;
-       PixbufRenderer *pr = rc->pr;
 
        rc_sync_actor(rc);
 }
 
-static void renderer_stereo_set(void *renderer, gint stereo_mode)
+static void rc_stereo_set(void *renderer, gint stereo_mode)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
 
        rc->stereo_mode = stereo_mode;
 }
 
-static void renderer_free(void *renderer)
+static void rc_free(void *renderer)
 {
        RendererClutter *rc = (RendererClutter *)renderer;
        GtkWidget *widget = gtk_bin_get_child(GTK_BIN(rc->pr));
 
-       renderer_remove_pending_updates(rc);
+       if (rc->display_pixbuf)
+               {
+               g_object_unref(rc->display_pixbuf);
+               }
+
+       rc_remove_pending_updates(rc);
 
        rc_overlay_free_all(rc);
-       
+
        if (widget)
                {
                /* widget still exists */
                clutter_actor_destroy(rc->group);
                if (clutter_group_get_n_children(CLUTTER_GROUP(rc->stage)) == 0)
                        {
-                       printf("destroy %p\n", rc->widget);
+                       DEBUG_1("destroy %p", rc->widget);
                        /* this was the last user */
                        gtk_widget_destroy(rc->widget);
                        }
                else
                        {
-                       printf("keep %p\n", rc->widget);
+                       DEBUG_1("keep %p", rc->widget);
                        g_object_unref(G_OBJECT(rc->widget));
                        }
                }
@@ -723,26 +765,25 @@ static void renderer_free(void *renderer)
 RendererFuncs *renderer_clutter_new(PixbufRenderer *pr)
 {
        RendererClutter *rc = g_new0(RendererClutter, 1);
-       
+
        rc->pr = pr;
-       
-       rc->f.area_changed = renderer_area_changed;
-       rc->f.update_pixbuf = renderer_update_pixbuf;
-       rc->f.free = renderer_free;
-       rc->f.update_zoom = renderer_update_zoom;
-       rc->f.invalidate_region = renderer_invalidate_region;
-       rc->f.scroll = renderer_scroll;
-       rc->f.update_sizes = renderer_update_sizes;
-
-
-       rc->f.overlay_add = renderer_overlay_add;
-       rc->f.overlay_set = renderer_overlay_set;
-       rc->f.overlay_get = renderer_overlay_get;
-       rc->f.overlay_draw = renderer_overlay_draw;
-
-       rc->f.stereo_set = renderer_stereo_set;
-       
-       
+
+       rc->f.area_changed = rc_area_changed;
+       rc->f.update_pixbuf = rc_update_pixbuf;
+       rc->f.free = rc_free;
+       rc->f.update_zoom = rc_update_zoom;
+       rc->f.invalidate_region = rc_invalidate_region;
+       rc->f.scroll = rc_scroll;
+       rc->f.update_viewport = rc_update_viewport;
+
+
+       rc->f.overlay_add = rc_overlay_add;
+       rc->f.overlay_set = rc_overlay_set;
+       rc->f.overlay_get = rc_overlay_get;
+
+       rc->f.stereo_set = rc_stereo_set;
+
+
        rc->stereo_mode = 0;
        rc->stereo_off_x = 0;
        rc->stereo_off_y = 0;
@@ -750,38 +791,42 @@ RendererFuncs *renderer_clutter_new(PixbufRenderer *pr)
        rc->idle_update = 0;
        rc->pending_updates = NULL;
 
-       rc->widget = gtk_bin_get_child(GTK_BIN(rc->pr));
-       
-       if (rc->widget)
-               {
-               if (!GTK_CLUTTER_IS_EMBED(rc->widget))
-                       {
-                       g_free(rc);
-                       DEBUG_0("pixbuf renderer has a child of other type than gtk_clutter_embed");
-                       return NULL;
-                       }
-               }
-       else 
-               {
-               rc->widget = gtk_clutter_embed_new();
+       rc->display_pixbuf = NULL;
+
+       rc->widget = gtk_bin_get_child(GTK_BIN(rc->pr));
+
+       if (rc->widget)
+               {
+               if (!GTK_CLUTTER_IS_EMBED(rc->widget))
+                       {
+                       g_free(rc);
+                       DEBUG_3("pixbuf renderer has a child of other type than gtk_clutter_embed");
+                       return NULL;
+                       }
+               }
+       else
+               {
+               rc->widget = gtk_clutter_embed_new();
                gtk_container_add(GTK_CONTAINER(rc->pr), rc->widget);
-               }
-               
+               }
+
        gtk_event_box_set_above_child (GTK_EVENT_BOX(rc->pr), TRUE);
-        rc->stage = gtk_clutter_embed_get_stage (GTK_CLUTTER_EMBED (rc->widget));
-        
-        rc->group = clutter_group_new();
-       clutter_container_add_actor(CLUTTER_CONTAINER(rc->stage), rc->group);
-       clutter_actor_set_clip_to_allocation(CLUTTER_ACTOR(rc->group), TRUE);
-  
-       rc->texture = clutter_texture_new ();
-       clutter_container_add_actor(CLUTTER_CONTAINER(rc->group), rc->texture);
-       rc_set_shader(clutter_texture_get_cogl_material(rc->texture));
-       g_object_ref(G_OBJECT(rc->widget));
-  
+       rc->stage = gtk_clutter_embed_get_stage (GTK_CLUTTER_EMBED (rc->widget));
+
+       rc->group = clutter_group_new();
+       clutter_container_add_actor(CLUTTER_CONTAINER(rc->stage), rc->group);
+       clutter_actor_set_clip_to_allocation(CLUTTER_ACTOR(rc->group), TRUE);
+
+       rc->texture = clutter_texture_new ();
+       clutter_container_add_actor(CLUTTER_CONTAINER(rc->group), rc->texture);
+
+       clutter_actor_set_content_scaling_filters(rc->texture, CLUTTER_SCALING_FILTER_TRILINEAR, CLUTTER_SCALING_FILTER_NEAREST);
+
+       g_object_ref(G_OBJECT(rc->widget));
+
        gtk_widget_show(rc->widget);
        return (RendererFuncs *) rc;
 }
 
-#endif 
+#endif
 /* vim: set shiftwidth=8 softtabstop=0 cindent cinoptions={1s: */