Vladimir Nadvornik wrote:
On po 25. května 2009, Omari Stephens wrote:
Nth time is the charm, apparently.  Patch (against r1711) attached.
 Doesn't seem to affect panview.


OK, if the fix is that simple we can add it now.

But the change below looks wrong. The color processing should be changed together with the pixbuf. That's why it is called from image_change_pixbuf.
With this change it is called earlier and probably affects previous image.
With delay_flip option there can be several seconds gap when the old
image is still displayed while the new one is loading.

How do you turn delay_flip on? I can't find it anymore. The "Smooth image flip" option doesn't seem to do anything, even when I'm in fullscreen mode.

Anyway, it didn't seem like image_change_pixbuf was being called for the layout_image images, but it seems I was wrong. Updated patch attached.

--xsdg
Index: src/img-view.c
===================================================================
--- src/img-view.c	(revision 1736)
+++ src/img-view.c	(working copy)
@@ -847,6 +847,10 @@
 	gtk_container_set_border_width(GTK_CONTAINER(vw->window), 0);
 
 	vw->imd = image_new(FALSE);
+	image_color_profile_set(vw->imd,
+				options->color_profile.input_type,
+				options->color_profile.use_image);
+	image_color_profile_set_use(vw->imd, options->color_profile.enabled);
 
 	image_background_set_color_from_options(vw->imd, FALSE);
 
Index: src/image.c
===================================================================
--- src/image.c	(revision 1736)
+++ src/image.c	(working copy)
@@ -1086,9 +1086,9 @@
 void image_change_pixbuf(ImageWindow *imd, GdkPixbuf *pixbuf, gdouble zoom, gboolean lazy)
 {
 
-
-	/* read_exif and similar functions can actually notice that the file has changed and triger a notification
-	that removes the pixbuf	from cache and unref it. Therefore we must ref it here before it is taken over by the renderer. */
+	/* read_exif and similar functions can actually notice that the file has changed and trigger
+	   a notification that removes the pixbuf from cache and unrefs it. Therefore we must ref it
+	   here before it is taken over by the renderer. */
 	if (pixbuf) g_object_ref(pixbuf); 
 	
 	if (imd->image_fd)
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Geeqie-devel mailing list
Geeqie-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geeqie-devel

Reply via email to