]> Pileus Git - ~andy/gtk/blobdiff - gdk-pixbuf/gdk-pixbuf-loader.c
Clarify misleading explanation of rowstride. (#119000)
[~andy/gtk] / gdk-pixbuf / gdk-pixbuf-loader.c
index 459da342aacc5949734ef022c3035f1d91dbe6b8..90635ed62a58d8a3814050f9514a86908d2609b4 100644 (file)
@@ -1,3 +1,4 @@
+/* -*- Mode: C; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 8 -*- */
 /* GdkPixbuf library - Progressive loader object
  *
  * Copyright (C) 1999 The Free Software Foundation
 #include <string.h>
 
 #include "gdk-pixbuf-private.h"
-#include "gdk-pixbuf-loader.h"
+#include "gdk-pixbuf-animation.h"
 #include "gdk-pixbuf-io.h"
+#include "gdk-pixbuf-loader.h"
 #include "gdk-pixbuf-marshal.h"
 
-#include <gobject/gsignal.h>
-
 enum {
-  AREA_UPDATED,
-  AREA_PREPARED,
-  FRAME_DONE,
-  ANIMATION_DONE,
-  CLOSED,
-  LAST_SIGNAL
+        SIZE_PREPARED,
+        AREA_PREPARED,
+        AREA_UPDATED,
+        CLOSED,
+        LAST_SIGNAL
 };
 
 
@@ -56,13 +55,16 @@ static guint    pixbuf_loader_signals[LAST_SIGNAL] = { 0 };
 
 typedef struct
 {
-  GdkPixbuf *pixbuf;
-  GdkPixbufAnimation *animation;
-  gboolean closed;
-  guchar header_buf[LOADER_HEADER_SIZE];
-  gint header_buf_offset;
-  GdkPixbufModule *image_module;
-  gpointer context;
+        GdkPixbufAnimation *animation;
+        gboolean closed;
+        guchar header_buf[LOADER_HEADER_SIZE];
+        gint header_buf_offset;
+        GdkPixbufModule *image_module;
+        gpointer context;
+        gint width;
+        gint height;
+        gboolean size_fixed;
+        gboolean needs_scale;
 } GdkPixbufLoaderPrivate;
 
 
@@ -70,7 +72,7 @@ typedef struct
  * gdk_pixbuf_loader_get_type:
  * @void:
  *
- * Registers the #GdkPixubfLoader class if necessary, and returns the type ID
+ * Registers the #GdkPixbufLoader class if necessary, and returns the type ID
  * associated to it.
  *
  * Return value: The type ID of the #GdkPixbufLoader class.
@@ -78,235 +80,216 @@ typedef struct
 GType
 gdk_pixbuf_loader_get_type (void)
 {
-  static GType loader_type = 0;
-  
-  if (!loader_type)
-    {
-      static const GTypeInfo loader_info = {
-        sizeof (GdkPixbufLoaderClass),
-        (GBaseInitFunc) NULL,
-        (GBaseFinalizeFunc) NULL,
-        (GClassInitFunc) gdk_pixbuf_loader_class_init,
-        NULL,           /* class_finalize */
-        NULL,           /* class_data */
-        sizeof (GdkPixbufLoader),
-        0,              /* n_preallocs */
-        (GInstanceInitFunc) gdk_pixbuf_loader_init
-      };
-
-      g_type_init ();
+        static GType loader_type = 0;
+  
+        if (!loader_type)
+                {
+                        static const GTypeInfo loader_info = {
+                                sizeof (GdkPixbufLoaderClass),
+                                (GBaseInitFunc) NULL,
+                                (GBaseFinalizeFunc) NULL,
+                                (GClassInitFunc) gdk_pixbuf_loader_class_init,
+                                NULL,           /* class_finalize */
+                                NULL,           /* class_data */
+                                sizeof (GdkPixbufLoader),
+                                0,              /* n_preallocs */
+                                (GInstanceInitFunc) gdk_pixbuf_loader_init
+                        };
       
-      loader_type = g_type_register_static (G_TYPE_OBJECT,
-                                            "GdkPixbufLoader",
-                                            &loader_info,
-                                            0);
-    }
+                        loader_type = g_type_register_static (G_TYPE_OBJECT,
+                                                              "GdkPixbufLoader",
+                                                              &loader_info,
+                                                              0);
+                }
   
-  return loader_type;
+        return loader_type;
 }
 
 static void
 gdk_pixbuf_loader_class_init (GdkPixbufLoaderClass *class)
 {
-  GObjectClass *object_class;
-  
-  object_class = (GObjectClass *) class;
-  
-  parent_class = g_type_class_peek_parent (class);
-  
-  object_class->finalize = gdk_pixbuf_loader_finalize;
-
-  pixbuf_loader_signals[AREA_PREPARED] =
-    g_signal_newc ("area_prepared",
-                   G_TYPE_FROM_CLASS (object_class),
-                   G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GdkPixbufLoaderClass, area_prepared),
-                   NULL,
-                   gdk_pixbuf_marshal_VOID__VOID,
-                   G_TYPE_NONE, 0);
-  
-  pixbuf_loader_signals[AREA_UPDATED] =
-    g_signal_newc ("area_updated",
-                   G_TYPE_FROM_CLASS (object_class),
-                   G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GdkPixbufLoaderClass, area_updated),
-                   NULL,
-                   gdk_pixbuf_marshal_VOID__INT_INT_INT_INT,
-                   G_TYPE_NONE, 4,
-                   G_TYPE_INT,
-                   G_TYPE_INT,
-                   G_TYPE_INT,
-                   G_TYPE_INT);
-  
-  pixbuf_loader_signals[FRAME_DONE] =
-    g_signal_newc ("frame_done",
-                   G_TYPE_FROM_CLASS (object_class),
-                   G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GdkPixbufLoaderClass, frame_done),
-                   NULL,
-                   gdk_pixbuf_marshal_VOID__POINTER,
-                   G_TYPE_NONE, 1,
-                   GDK_TYPE_PIXBUF_FRAME);
-  
-  pixbuf_loader_signals[ANIMATION_DONE] =
-    g_signal_newc ("animation_done",
-                   G_TYPE_FROM_CLASS (object_class),
-                   G_SIGNAL_RUN_LAST,                   
-                   G_STRUCT_OFFSET (GdkPixbufLoaderClass, animation_done),
-                   NULL,
-                   gdk_pixbuf_marshal_VOID__VOID,
-                   G_TYPE_NONE, 0);
-  
-  pixbuf_loader_signals[CLOSED] =
-    g_signal_newc ("closed",
-                   G_TYPE_FROM_CLASS (object_class),
-                   G_SIGNAL_RUN_LAST,
-                   G_STRUCT_OFFSET (GdkPixbufLoaderClass, closed),
-                   NULL,
-                   gdk_pixbuf_marshal_VOID__VOID,
-                   G_TYPE_NONE, 0);
+        GObjectClass *object_class;
+  
+        object_class = (GObjectClass *) class;
+  
+        parent_class = g_type_class_peek_parent (class);
+  
+        object_class->finalize = gdk_pixbuf_loader_finalize;
+
+        pixbuf_loader_signals[SIZE_PREPARED] =
+                g_signal_new ("size_prepared",
+                              G_TYPE_FROM_CLASS (object_class),
+                              G_SIGNAL_RUN_LAST,
+                              G_STRUCT_OFFSET (GdkPixbufLoaderClass, size_prepared),
+                              NULL, NULL,
+                              gdk_pixbuf_marshal_VOID__INT_INT,
+                              G_TYPE_NONE, 2, 
+                              G_TYPE_INT,
+                              G_TYPE_INT);
+  
+        pixbuf_loader_signals[AREA_PREPARED] =
+                g_signal_new ("area_prepared",
+                              G_TYPE_FROM_CLASS (object_class),
+                              G_SIGNAL_RUN_LAST,
+                              G_STRUCT_OFFSET (GdkPixbufLoaderClass, area_prepared),
+                              NULL, NULL,
+                              gdk_pixbuf_marshal_VOID__VOID,
+                              G_TYPE_NONE, 0);
+  
+        pixbuf_loader_signals[AREA_UPDATED] =
+                g_signal_new ("area_updated",
+                              G_TYPE_FROM_CLASS (object_class),
+                              G_SIGNAL_RUN_LAST,
+                              G_STRUCT_OFFSET (GdkPixbufLoaderClass, area_updated),
+                              NULL, NULL,
+                              gdk_pixbuf_marshal_VOID__INT_INT_INT_INT,
+                              G_TYPE_NONE, 4,
+                              G_TYPE_INT,
+                              G_TYPE_INT,
+                              G_TYPE_INT,
+                              G_TYPE_INT);
+  
+        pixbuf_loader_signals[CLOSED] =
+                g_signal_new ("closed",
+                              G_TYPE_FROM_CLASS (object_class),
+                              G_SIGNAL_RUN_LAST,
+                              G_STRUCT_OFFSET (GdkPixbufLoaderClass, closed),
+                              NULL, NULL,
+                              gdk_pixbuf_marshal_VOID__VOID,
+                              G_TYPE_NONE, 0);
 }
 
 static void
 gdk_pixbuf_loader_init (GdkPixbufLoader *loader)
 {
-  GdkPixbufLoaderPrivate *priv;
+        GdkPixbufLoaderPrivate *priv;
   
-  priv = g_new0 (GdkPixbufLoaderPrivate, 1);
-  loader->priv = priv;
+        priv = g_new0 (GdkPixbufLoaderPrivate, 1);
+        loader->priv = priv;
 }
 
 static void
 gdk_pixbuf_loader_finalize (GObject *object)
 {
-  GdkPixbufLoader *loader;
-  GdkPixbufLoaderPrivate *priv = NULL;
+        GdkPixbufLoader *loader;
+        GdkPixbufLoaderPrivate *priv = NULL;
   
-  loader = GDK_PIXBUF_LOADER (object);
-  priv = loader->priv;
+        loader = GDK_PIXBUF_LOADER (object);
+        priv = loader->priv;
 
-  if (!priv->closed)
-    g_warning ("GdkPixbufLoader finalized without calling gdk_pixbuf_loader_close() - this is not allowed. You must explicitly end the data stream to the loader before dropping the last reference.");
+        if (!priv->closed)
+                g_warning ("GdkPixbufLoader finalized without calling gdk_pixbuf_loader_close() - this is not allowed. You must explicitly end the data stream to the loader before dropping the last reference.");
   
-  if (priv->animation)
-    gdk_pixbuf_animation_unref (priv->animation);
-
-  if (priv->pixbuf)
-    gdk_pixbuf_unref (priv->pixbuf);
+        if (priv->animation)
+                g_object_unref (priv->animation);
   
-  g_free (priv);
+        g_free (priv);
   
-  G_OBJECT_CLASS (parent_class)->finalize (object);
+        G_OBJECT_CLASS (parent_class)->finalize (object);
 }
 
-static void
-gdk_pixbuf_loader_prepare (GdkPixbuf *pixbuf,
-                          gpointer   loader)
+/**
+ * gdk_pixbuf_loader_set_size:
+ * @loader: A pixbuf loader.
+ * @width: The desired width of the image being loaded.
+ * @height: The desired height of the image being loaded.
+ *
+ * Causes the image to be scaled while it is loaded. The desired
+ * image size can be determined relative to the original size of
+ * the image by calling gdk_pixbuf_loader_set_size() from a
+ * signal handler for the ::size_prepared signal.
+ *
+ * Attempts to set the desired image size  are ignored after the 
+ * emission of the ::size_prepared signal.
+ *
+ * Since: 2.2
+ */
+void 
+gdk_pixbuf_loader_set_size (GdkPixbufLoader *loader,
+                           gint             width,
+                           gint             height)
 {
-  GdkPixbufLoaderPrivate *priv = NULL;
-  
-  priv = GDK_PIXBUF_LOADER (loader)->priv;
-  gdk_pixbuf_ref (pixbuf);
-
-  g_assert (priv->pixbuf == NULL);
-  
-  priv->pixbuf = pixbuf;
-  g_signal_emit (G_OBJECT (loader), pixbuf_loader_signals[AREA_PREPARED], 0);
+        GdkPixbufLoaderPrivate *priv = GDK_PIXBUF_LOADER (loader)->priv;
+        g_return_if_fail (width > 0 && height > 0);
+
+        if (!priv->size_fixed) 
+                {
+                        priv->width = width;
+                        priv->height = height;
+                }
 }
 
 static void
-gdk_pixbuf_loader_update (GdkPixbuf *pixbuf,
-                         guint      x,
-                         guint      y,
-                         guint      width,
-                         guint      height,
-                         gpointer   loader)
+gdk_pixbuf_loader_size_func (gint *width, gint *height, gpointer loader)
 {
-  GdkPixbufLoaderPrivate *priv = NULL;
-  
-  priv = GDK_PIXBUF_LOADER (loader)->priv;
-  
-  g_signal_emit (G_OBJECT (loader),
-                 pixbuf_loader_signals[AREA_UPDATED],
-                 0,
-                 x, y,
-                 /* sanity check in here.  Defend against an errant loader */
-                 MIN (width, gdk_pixbuf_get_width (priv->pixbuf)),
-                 MIN (height, gdk_pixbuf_get_height (priv->pixbuf)));
+        GdkPixbufLoaderPrivate *priv = GDK_PIXBUF_LOADER (loader)->priv;
+
+        /* allow calling gdk_pixbuf_loader_set_size() before the signal */
+        if (priv->width == 0 && priv->height == 0) 
+                {
+                        priv->width = *width;
+                        priv->height = *height;
+                }
+
+        g_signal_emit (loader, pixbuf_loader_signals[SIZE_PREPARED], 0, *width, *height);
+        priv->size_fixed = TRUE;
+
+        *width = priv->width;
+        *height = priv->height;
 }
 
 static void
-gdk_pixbuf_loader_frame_done (GdkPixbufFrame *frame,
-                             gpointer        loader)
+gdk_pixbuf_loader_prepare (GdkPixbuf          *pixbuf,
+                           GdkPixbufAnimation *anim,
+                          gpointer            loader)
 {
-  GdkPixbufLoaderPrivate *priv = NULL;
-  
-  priv = GDK_PIXBUF_LOADER (loader)->priv;
-  
-  priv->pixbuf = NULL;
-  
-  if (priv->animation == NULL)
-    {
-      priv->animation = g_object_new (GDK_TYPE_PIXBUF_ANIMATION, NULL);
-      
-      priv->animation->n_frames = 0;
-      priv->animation->width  = gdk_pixbuf_get_width  (frame->pixbuf) + frame->x_offset;
-      priv->animation->height = gdk_pixbuf_get_height (frame->pixbuf) + frame->y_offset;
-    }
-  else
-    {
-      int w, h;
-      
-      /* update bbox size */
-      w = gdk_pixbuf_get_width (frame->pixbuf) + frame->x_offset;
-      h = gdk_pixbuf_get_height (frame->pixbuf) + frame->y_offset;
-      
-      if (w > priv->animation->width) {
-       priv->animation->width = w;
-      }
-      if (h > priv->animation->height) {
-       priv->animation->height = h;
-      }
-    }
-  
-  priv->animation->frames = g_list_append (priv->animation->frames, frame);
-  priv->animation->n_frames++;
-  g_signal_emit (G_OBJECT (loader),
-                 pixbuf_loader_signals[FRAME_DONE],
-                 0,
-                 frame);
+        GdkPixbufLoaderPrivate *priv = GDK_PIXBUF_LOADER (loader)->priv;
+        g_return_if_fail (pixbuf != NULL);
+
+        if (!priv->size_fixed) 
+                {
+                        /* Defend against lazy loaders which don't call size_func */
+                        gint width = gdk_pixbuf_get_width (pixbuf);
+                        gint height = gdk_pixbuf_get_height (pixbuf);
+                        
+                        gdk_pixbuf_loader_size_func (&width, &height, loader);
+                }
+
+        priv->needs_scale = FALSE;
+        if (priv->width > 0 && priv->height > 0 &&
+            (priv->width != gdk_pixbuf_get_width (pixbuf) ||
+             priv->height != gdk_pixbuf_get_height (pixbuf)))
+                priv->needs_scale = TRUE;
+
+        if (anim)
+                g_object_ref (anim);
+        else
+                anim = gdk_pixbuf_non_anim_new (pixbuf);
+  
+        priv->animation = anim;
+  
+        if (!priv->needs_scale)
+                g_signal_emit (loader, pixbuf_loader_signals[AREA_PREPARED], 0);
 }
 
 static void
-gdk_pixbuf_loader_animation_done (GdkPixbuf *pixbuf,
-                                 gpointer   loader)
+gdk_pixbuf_loader_update (GdkPixbuf *pixbuf,
+                         gint       x,
+                         gint       y,
+                         gint       width,
+                         gint       height,
+                         gpointer   loader)
 {
-  GdkPixbufLoaderPrivate *priv = NULL;
-  GdkPixbufFrame    *frame;
-  GList *current = NULL;
-  gint h, w;
-  
-  priv = GDK_PIXBUF_LOADER (loader)->priv;
-  priv->pixbuf = NULL;
-  
-  current = gdk_pixbuf_animation_get_frames (priv->animation);
-  
-  while (current)
-    {
-      frame = (GdkPixbufFrame *) current->data;
-      
-      /* update bbox size */
-      w = gdk_pixbuf_get_width (frame->pixbuf) + frame->x_offset;
-      h = gdk_pixbuf_get_height (frame->pixbuf) + frame->y_offset;
-      
-      if (w > priv->animation->width)
-       priv->animation->width = w;
-      if (h > priv->animation->height)
-       priv->animation->height = h;
-      current = current->next;
-    }
-  
-  g_signal_emit (G_OBJECT (loader), pixbuf_loader_signals[ANIMATION_DONE], 0);
+        GdkPixbufLoaderPrivate *priv = GDK_PIXBUF_LOADER (loader)->priv;
+  
+        if (!priv->needs_scale)
+                g_signal_emit (loader,
+                               pixbuf_loader_signals[AREA_UPDATED],
+                               0,
+                               x, y,
+                               /* sanity check in here.  Defend against an errant loader */
+                               MIN (width, gdk_pixbuf_animation_get_width (priv->animation)),
+                               MIN (height, gdk_pixbuf_animation_get_height (priv->animation)));
 }
 
 static gint
@@ -314,79 +297,77 @@ gdk_pixbuf_loader_load_module (GdkPixbufLoader *loader,
                                const char      *image_type,
                                GError         **error)
 {
-  GdkPixbufLoaderPrivate *priv = loader->priv;
-
-  if (image_type)
-    {
-      priv->image_module = gdk_pixbuf_get_named_module (image_type,
+        GdkPixbufLoaderPrivate *priv = loader->priv;
+
+        if (image_type)
+                {
+                        priv->image_module = _gdk_pixbuf_get_named_module (image_type,
+                                                                           error);
+                }
+        else
+                {
+                        priv->image_module = _gdk_pixbuf_get_module (priv->header_buf,
+                                                                     priv->header_buf_offset,
+                                                                     NULL,
+                                                                     error);
+                }
+  
+        if (priv->image_module == NULL)
+                return 0;
+  
+        if (priv->image_module->module == NULL)
+                if (!_gdk_pixbuf_load_module (priv->image_module, error))
+                        return 0;
+  
+        if (priv->image_module->module == NULL)
+                return 0;
+  
+        if ((priv->image_module->begin_load == NULL) ||
+            (priv->image_module->stop_load == NULL) ||
+            (priv->image_module->load_increment == NULL))
+                {
+                        g_set_error (error,
+                                     GDK_PIXBUF_ERROR,
+                                     GDK_PIXBUF_ERROR_UNSUPPORTED_OPERATION,
+                                     _("Incremental loading of image type '%s' is not supported"),
+                                     priv->image_module->module_name);
+
+                        return 0;
+                }
+
+        priv->context = priv->image_module->begin_load (gdk_pixbuf_loader_size_func,
+                                                        gdk_pixbuf_loader_prepare,
+                                                        gdk_pixbuf_loader_update,
+                                                        loader,
                                                         error);
-    }
-  else
-    {
-      g_return_val_if_fail (priv->header_buf_offset > 0, 0);
-      priv->image_module = gdk_pixbuf_get_module (priv->header_buf,
-                                                  priv->header_buf_offset,
-                                                  NULL,
-                                                  error);
-    }
-  
-  if (priv->image_module == NULL)
-    return 0;
-  
-  if (priv->image_module->module == NULL)
-    if (!gdk_pixbuf_load_module (priv->image_module, error))
-      return 0;
-  
-  if (priv->image_module->module == NULL)
-    return 0;
-  
-  if ((priv->image_module->begin_load == NULL) ||
-      (priv->image_module->stop_load == NULL) ||
-      (priv->image_module->load_increment == NULL))
-    {
-      g_set_error (error,
-                   GDK_PIXBUF_ERROR,
-                   GDK_PIXBUF_ERROR_UNSUPPORTED_OPERATION,
-                   _("Incremental loading of image type '%s' is not supported"),
-                   image_type);
-
-      return 0;
-    }
-  
-  priv->context = priv->image_module->begin_load (gdk_pixbuf_loader_prepare,
-                                                 gdk_pixbuf_loader_update,
-                                                 gdk_pixbuf_loader_frame_done,
-                                                 gdk_pixbuf_loader_animation_done,
-                                                 loader,
-                                                  error);
-  
-  if (priv->context == NULL)
-    {
-      /* Defense against broken loaders; DO NOT take this as a GError
-       * example
-       */
-      if (error && *error == NULL)
-        {
-          g_warning ("Bug! loader '%s' didn't set an error on failure",
-                     priv->image_module->module_name);
-          g_set_error (error,
-                       GDK_PIXBUF_ERROR,
-                       GDK_PIXBUF_ERROR_FAILED,
-                       _("Internal error: Image loader module '%s'"
-                         " failed to begin loading an image, but didn't"
-                         " give a reason for the failure"),
-                       priv->image_module->module_name);
-
-        }
+  
+        if (priv->context == NULL)
+                {
+                        /* Defense against broken loaders; DO NOT take this as a GError
+                         * example
+                         */
+                        if (error && *error == NULL)
+                                {
+                                        g_warning ("Bug! loader '%s' didn't set an error on failure",
+                                                   priv->image_module->module_name);
+                                        g_set_error (error,
+                                                     GDK_PIXBUF_ERROR,
+                                                     GDK_PIXBUF_ERROR_FAILED,
+                                                     _("Internal error: Image loader module '%s'"
+                                                       " failed to begin loading an image, but didn't"
+                                                       " give a reason for the failure"),
+                                                     priv->image_module->module_name);
+
+                                }
       
-      return 0;
-    }
+                        return 0;
+                }
   
-  if (priv->header_buf_offset
-      && priv->image_module->load_increment (priv->context, priv->header_buf, priv->header_buf_offset, error))
-    return priv->header_buf_offset;
+        if (priv->header_buf_offset
+            && priv->image_module->load_increment (priv->context, priv->header_buf, priv->header_buf_offset, error))
+                return priv->header_buf_offset;
   
-  return 0;
+        return 0;
 }
 
 static int
@@ -395,21 +376,21 @@ gdk_pixbuf_loader_eat_header_write (GdkPixbufLoader *loader,
                                    gsize            count,
                                     GError         **error)
 {
-  gint n_bytes;
-  GdkPixbufLoaderPrivate *priv = loader->priv;
+        gint n_bytes;
+        GdkPixbufLoaderPrivate *priv = loader->priv;
   
-  n_bytes = MIN(LOADER_HEADER_SIZE - priv->header_buf_offset, count);
-  memcpy (priv->header_buf + priv->header_buf_offset, buf, n_bytes);
+        n_bytes = MIN(LOADER_HEADER_SIZE - priv->header_buf_offset, count);
+        memcpy (priv->header_buf + priv->header_buf_offset, buf, n_bytes);
   
-  priv->header_buf_offset += n_bytes;
+        priv->header_buf_offset += n_bytes;
   
-  if (priv->header_buf_offset >= LOADER_HEADER_SIZE)
-    {
-      if (gdk_pixbuf_loader_load_module (loader, NULL, error) == 0)
-       return 0;
-    }
+        if (priv->header_buf_offset >= LOADER_HEADER_SIZE)
+                {
+                        if (gdk_pixbuf_loader_load_module (loader, NULL, error) == 0)
+                                return 0;
+                }
   
-  return n_bytes;
+        return n_bytes;
 }
 
 /**
@@ -420,13 +401,13 @@ gdk_pixbuf_loader_eat_header_write (GdkPixbufLoader *loader,
  * @error: return location for errors
  *
  * This will cause a pixbuf loader to parse the next @count bytes of
- * an image.  It will return TRUE if the data was loaded successfully,
- * and FALSE if an error occurred.  In the latter case, the loader
- * will be closed, and will not accept further writes. If FALSE is
+ * an image.  It will return %TRUE if the data was loaded successfully,
+ * and %FALSE if an error occurred.  In the latter case, the loader
+ * will be closed, and will not accept further writes. If %FALSE is
  * returned, @error will be set to an error from the #GDK_PIXBUF_ERROR
  * or #G_FILE_ERROR domains.
  *
- * Return value: #TRUE if the write was successful, or #FALSE if the loader
+ * Return value: %TRUE if the write was successful, or %FALSE if the loader
  * cannot parse the buffer.
  **/
 gboolean
@@ -435,54 +416,55 @@ gdk_pixbuf_loader_write (GdkPixbufLoader *loader,
                         gsize            count,
                          GError         **error)
 {
-  GdkPixbufLoaderPrivate *priv;
-  
-  g_return_val_if_fail (loader != NULL, FALSE);
-  g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), FALSE);
+        GdkPixbufLoaderPrivate *priv;
   
-  g_return_val_if_fail (buf != NULL, FALSE);
-  g_return_val_if_fail (count >= 0, FALSE);
+        g_return_val_if_fail (loader != NULL, FALSE);
+        g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), FALSE);
   
-  priv = loader->priv;
+        g_return_val_if_fail (buf != NULL, FALSE);
+        g_return_val_if_fail (count >= 0, FALSE);
+        g_return_val_if_fail (error == NULL || *error == NULL, FALSE);
   
-  /* we expect it's not to be closed */
-  g_return_val_if_fail (priv->closed == FALSE, FALSE);
+        priv = loader->priv;
+
+        /* we expect it's not to be closed */
+        g_return_val_if_fail (priv->closed == FALSE, FALSE);
   
-  if (priv->image_module == NULL)
-    {
-      gint eaten;
+        if (priv->image_module == NULL)
+                {
+                        gint eaten;
       
-      eaten = gdk_pixbuf_loader_eat_header_write(loader, buf, count, error);
-      if (eaten <= 0)
-       return FALSE;
+                        eaten = gdk_pixbuf_loader_eat_header_write (loader, buf, count, error);
+                        if (eaten <= 0)
+                                return FALSE;
       
-      count -= eaten;
-      buf += eaten;
-    }
-  
-  if (count > 0 && priv->image_module->load_increment)
-    {
-      gboolean retval;
-      retval = priv->image_module->load_increment (priv->context, buf, count,
-                                                   error);
-      if (!retval && error && *error == NULL)
-        {
-          /* Fix up busted image loader */
-          g_warning ("Bug! loader '%s' didn't set an error on failure",
-                     priv->image_module->module_name);
-          g_set_error (error,
-                       GDK_PIXBUF_ERROR,
-                       GDK_PIXBUF_ERROR_FAILED,
-                       _("Internal error: Image loader module '%s'"
-                         " failed to begin loading an image, but didn't"
-                         " give a reason for the failure"),
-                       priv->image_module->module_name);
-        }
-
-      return retval;
-    }
+                        count -= eaten;
+                        buf += eaten;
+                }
+  
+        if (count > 0 && priv->image_module->load_increment)
+                {
+                        gboolean retval;
+                        retval = priv->image_module->load_increment (priv->context, buf, count,
+                                                                     error);
+                        if (!retval && error && *error == NULL)
+                                {
+                                        /* Fix up busted image loader */
+                                        g_warning ("Bug! loader '%s' didn't set an error on failure",
+                                                   priv->image_module->module_name);
+                                        g_set_error (error,
+                                                     GDK_PIXBUF_ERROR,
+                                                     GDK_PIXBUF_ERROR_FAILED,
+                                                     _("Internal error: Image loader module '%s'"
+                                                       " failed to begin loading an image, but didn't"
+                                                       " give a reason for the failure"),
+                                                     priv->image_module->module_name);
+                                }
+
+                        return retval;
+                }
       
-  return TRUE;
+        return TRUE;
 }
 
 /**
@@ -495,13 +477,20 @@ gdk_pixbuf_loader_write (GdkPixbufLoader *loader,
 GdkPixbufLoader *
 gdk_pixbuf_loader_new (void)
 {
-  return g_object_new (GDK_TYPE_PIXBUF_LOADER, NULL);
+        return g_object_new (GDK_TYPE_PIXBUF_LOADER, NULL);
 }
 
 /**
  * gdk_pixbuf_loader_new_with_type:
+ * @image_type: name of the image format to be loaded with the image
+ * @error: return location for an allocated #GError, or %NULL to ignore errors
  *
- * Creates a new pixbuf loader object.
+ * Creates a new pixbuf loader object that always attempts to parse
+ * image data as if it were an image of type @image_type, instead of
+ * identifying the type automatically. Useful if you want an error if
+ * the image isn't the expected type, for loading image formats
+ * that can't be reliably identified by looking at the data, or if
+ * the user manually forces a specific type.
  *
  * Return value: A newly-created pixbuf loader.
  **/
@@ -509,90 +498,142 @@ GdkPixbufLoader *
 gdk_pixbuf_loader_new_with_type (const char *image_type,
                                  GError    **error)
 {
-  GdkPixbufLoader *retval;
-  GError *tmp;
-  
-  retval = g_object_new (GDK_TYPE_PIXBUF_LOADER, NULL);
+        GdkPixbufLoader *retval;
+        GError *tmp;
+        g_return_val_if_fail (error == NULL || *error == NULL, NULL);
+  
+        retval = g_object_new (GDK_TYPE_PIXBUF_LOADER, NULL);
+
+        tmp = NULL;
+        gdk_pixbuf_loader_load_module (retval, image_type, &tmp);
+        if (tmp != NULL)
+                {
+                        g_propagate_error (error, tmp);
+                        g_object_unref (retval);
+                        return NULL;
+                }
+
+        return retval;
+}
 
-  tmp = NULL;
-  gdk_pixbuf_loader_load_module(retval, image_type, &tmp);
-  if (tmp != NULL)
-    {
-      g_propagate_error (error, tmp);
-      g_object_unref (G_OBJECT (retval));
-      return NULL;
-    }
+/**
+ * gdk_pixbuf_loader_new_with_mime_type:
+ * @mime_type: the mime type to be loaded 
+ * @error: return location for an allocated #GError, or %NULL to ignore errors
+ *
+ * Creates a new pixbuf loader object that always attempts to parse
+ * image data as if it were an image of mime type @mime_type, instead of
+ * identifying the type automatically. Useful if you want an error if
+ * the image isn't the expected mime type, for loading image formats
+ * that can't be reliably identified by looking at the data, or if
+ * the user manually forces a specific mime type.
+ *
+ * Return value: A newly-created pixbuf loader.
+ * Since: 2.4
+ **/
+GdkPixbufLoader *
+gdk_pixbuf_loader_new_with_mime_type (const char *mime_type,
+                                      GError    **error)
+{
+        const char * image_type = NULL;
+        char ** mimes;
+
+        GdkPixbufLoader *retval;
+        GError *tmp;
+  
+        GSList * formats;
+        GdkPixbufFormat *info;
+        int i, j, length;
+
+        formats = gdk_pixbuf_get_formats ();
+        length = g_slist_length (formats);
+
+        for (i = 0; i < length && image_type == NULL; i++) {
+                info = (GdkPixbufFormat *)g_slist_nth_data (formats, i);
+                mimes = info->mime_types;
+                
+                for (j = 0; mimes[j] != NULL; j++)
+                        if (g_ascii_strcasecmp (mimes[i], mime_type)) {
+                                image_type = info->name;
+                                break;
+                        }
+        }
+
+        g_slist_free (formats);
 
-  return retval;
+        retval = g_object_new (GDK_TYPE_PIXBUF_LOADER, NULL);
+
+        tmp = NULL;
+        gdk_pixbuf_loader_load_module(retval, image_type, &tmp);
+        if (tmp != NULL)
+                {
+                        g_propagate_error (error, tmp);
+                        g_object_unref (retval);
+                        return NULL;
+                }
+
+        return retval;
 }
 
 /**
  * gdk_pixbuf_loader_get_pixbuf:
  * @loader: A pixbuf loader.
  *
- * Queries the GdkPixbuf that a pixbuf loader is currently creating.
- * In general it only makes sense to call this function afer the
+ * Queries the #GdkPixbuf that a pixbuf loader is currently creating.
+ * In general it only makes sense to call this function after the
  * "area_prepared" signal has been emitted by the loader; this means
  * that enough data has been read to know the size of the image that
  * will be allocated.  If the loader has not received enough data via
- * gdk_pixbuf_loader_write(), then this function returns NULL.  The
+ * gdk_pixbuf_loader_write(), then this function returns %NULL.  The
  * returned pixbuf will be the same in all future calls to the loader,
- * so simply calling gdk_pixbuf_ref() should be sufficient to continue
+ * so simply calling g_object_ref() should be sufficient to continue
  * using it.  Additionally, if the loader is an animation, it will
- * return the first frame of the animation.
+ * return the "static image" of the animation
+ * (see gdk_pixbuf_animation_get_static_image()).
  * 
- * Return value: The GdkPixbuf that the loader is creating, or NULL if not
+ * Return value: The #GdkPixbuf that the loader is creating, or %NULL if not
  * enough data has been read to determine how to create the image buffer.
  **/
 GdkPixbuf *
 gdk_pixbuf_loader_get_pixbuf (GdkPixbufLoader *loader)
 {
-  GdkPixbufLoaderPrivate *priv;
+        GdkPixbufLoaderPrivate *priv;
   
-  g_return_val_if_fail (loader != NULL, NULL);
-  g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), NULL);
+        g_return_val_if_fail (loader != NULL, NULL);
+        g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), NULL);
   
-  priv = loader->priv;
+        priv = loader->priv;
 
-  if (priv->animation)
-    {
-      GList *list;
-      
-      list = gdk_pixbuf_animation_get_frames (priv->animation);
-      if (list != NULL)
-        {
-          GdkPixbufFrame *frame = list->data;
-          
-          return gdk_pixbuf_frame_get_pixbuf (frame);
-        }
-    }
-  
-  return priv->pixbuf;
+        if (priv->animation)
+                return gdk_pixbuf_animation_get_static_image (priv->animation);
+        else
+                return NULL;
 }
 
 /**
  * gdk_pixbuf_loader_get_animation:
  * @loader: A pixbuf loader
  *
- * Queries the GdkPixbufAnimation that a pixbuf loader is currently creating.
- * In general it only makes sense to call this function afer the "area_prepared"
- * signal has been emitted by the loader.  If the image is not an animation,
- * then it will return NULL.
+ * Queries the #GdkPixbufAnimation that a pixbuf loader is currently creating.
+ * In general it only makes sense to call this function after the "area_prepared"
+ * signal has been emitted by the loader. If the loader doesn't have enough
+ * bytes yet (hasn't emitted the "area_prepared" signal) this function will 
+ * return %NULL.
  *
- * Return value: The GdkPixbufAnimation that the loader is loading, or NULL if
+ * Return value: The #GdkPixbufAnimation that the loader is loading, or %NULL if
  not enough data has been read to determine the information.
 **/
 GdkPixbufAnimation *
 gdk_pixbuf_loader_get_animation (GdkPixbufLoader *loader)
 {
-  GdkPixbufLoaderPrivate *priv;
+        GdkPixbufLoaderPrivate *priv;
   
-  g_return_val_if_fail (loader != NULL, NULL);
-  g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), NULL);
+        g_return_val_if_fail (loader != NULL, NULL);
+        g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), NULL);
   
-  priv = loader->priv;
+        priv = loader->priv;
   
-  return priv->animation;
+        return priv->animation;
 }
 
 /**
@@ -604,7 +645,7 @@ gdk_pixbuf_loader_get_animation (GdkPixbufLoader *loader)
  * gdk_pixbuf_loader_write() will occur, so that it can free its
  * internal loading structures. Also, tries to parse any data that
  * hasn't yet been parsed; if the remaining data is partial or
- * corrupt, an error will be returned.  If FALSE is returned, @error
+ * corrupt, an error will be returned.  If %FALSE is returned, @error
  * will be set to an error from the #GDK_PIXBUF_ERROR or #G_FILE_ERROR
  * domains. If you're just cancelling a load rather than expecting it
  * to be finished, passing %NULL for @error to ignore it is
@@ -617,27 +658,93 @@ gboolean
 gdk_pixbuf_loader_close (GdkPixbufLoader *loader,
                          GError         **error)
 {
-  GdkPixbufLoaderPrivate *priv;
-  gboolean retval = TRUE;
-  
-  g_return_val_if_fail (loader != NULL, TRUE);
-  g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), TRUE);
-  
-  priv = loader->priv;
-  
-  /* we expect it's not closed */
-  g_return_val_if_fail (priv->closed == FALSE, TRUE);
-  
-  /* We have less the 128 bytes in the image.  Flush it, and keep going. */
-  if (priv->image_module == NULL)
-    gdk_pixbuf_loader_load_module (loader, NULL, NULL);
-  
-  if (priv->image_module && priv->image_module->stop_load)
-    retval = priv->image_module->stop_load (priv->context, error);
+        GdkPixbufLoaderPrivate *priv;
+        gboolean retval = TRUE;
+  
+        g_return_val_if_fail (loader != NULL, TRUE);
+        g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), TRUE);
+        g_return_val_if_fail (error == NULL || *error == NULL, TRUE);
+  
+        priv = loader->priv;
+  
+        /* we expect it's not closed */
+        g_return_val_if_fail (priv->closed == FALSE, TRUE);
+  
+        /* We have less the 128 bytes in the image.  Flush it, and keep going. */
+        if (priv->image_module == NULL)
+                {
+                        GError *tmp = NULL;
+                        gdk_pixbuf_loader_load_module (loader, NULL, &tmp);
+                        if (tmp != NULL)
+                                {
+                                        g_propagate_error (error, tmp);
+                                        retval = FALSE;
+                                }
+                }  
+
+        if (priv->image_module && priv->image_module->stop_load && priv->context) 
+                {
+                        if (!priv->image_module->stop_load (priv->context, error))
+                                retval = FALSE;
+                }
+  
+        priv->closed = TRUE;
+
+        if (priv->needs_scale) 
+                {
+                        GdkPixbuf *tmp, *pixbuf;
+                        
+                        tmp = gdk_pixbuf_animation_get_static_image (priv->animation);
+                        g_object_ref (tmp);
+                        pixbuf = gdk_pixbuf_new (GDK_COLORSPACE_RGB, tmp->has_alpha, 8, priv->width, priv->height);
+                        g_object_unref (priv->animation);
+                        priv->animation = gdk_pixbuf_non_anim_new (pixbuf);
+                        g_object_unref (pixbuf);
+                        g_signal_emit (loader, pixbuf_loader_signals[AREA_PREPARED], 0);
+                        gdk_pixbuf_scale (tmp, pixbuf, 0, 0, priv->width, priv->height, 0, 0,
+                                          (double) priv->width / tmp->width,
+                                          (double) priv->height / tmp->height,
+                                          GDK_INTERP_BILINEAR); 
+                        g_object_unref (tmp);
+                        
+                        g_signal_emit (loader, pixbuf_loader_signals[AREA_UPDATED], 0, 
+                                       0, 0, priv->width, priv->height);
+                }
+
+        
+        g_signal_emit (loader, pixbuf_loader_signals[CLOSED], 0);
+
+        return retval;
+}
+
+/**
+ * gdk_pixbuf_loader_get_format:
+ * @loader: A pixbuf loader.
+ *
+ * Obtains the available information about the format of the 
+ * currently loading image file.
+ *
+ * Returns: A #GdkPixbufFormat or %NULL. The return value is owned 
+ * by GdkPixbuf and should not be freed.
+ * 
+ * Since: 2.2
+ */
+GdkPixbufFormat *
+gdk_pixbuf_loader_get_format (GdkPixbufLoader *loader)
+{
+        GdkPixbufLoaderPrivate *priv;
   
-  priv->closed = TRUE;
+        g_return_val_if_fail (loader != NULL, NULL);
+        g_return_val_if_fail (GDK_IS_PIXBUF_LOADER (loader), NULL);
   
-  g_signal_emit (G_OBJECT (loader), pixbuf_loader_signals[CLOSED], 0);
+        priv = loader->priv;
 
-  return retval;
+        if (priv->image_module)
+                return _gdk_pixbuf_get_format (priv->image_module);
+        else
+                return NULL;
 }
+
+
+
+