[gimp] Make gimp_image_get_layer, channel, vectors_by_name, tattoo() more readable



commit d1f767d9b6a8e05470314d6821ffd41225045c0f
Author: Michael Natterer <mitch gimp org>
Date:   Tue Sep 8 10:23:47 2009 +0200

    Make gimp_image_get_layer,channel,vectors_by_name,tattoo() more readable
    
    Add local GimpItemStack variables instead of having the functions'
    entire code in one line.

 app/core/gimpimage.c |   42 ++++++++++++++++++++++++++++++------------
 1 files changed, 30 insertions(+), 12 deletions(-)
---
diff --git a/app/core/gimpimage.c b/app/core/gimpimage.c
index 7eedda2..ce40c77 100644
--- a/app/core/gimpimage.c
+++ b/app/core/gimpimage.c
@@ -2885,63 +2885,81 @@ GimpLayer *
 gimp_image_get_layer_by_tattoo (const GimpImage *image,
                                 GimpTattoo       tattoo)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
 
-  return GIMP_LAYER (gimp_item_stack_get_item_by_tattoo (GIMP_ITEM_STACK (image->layers),
-                                                         tattoo));
+  stack = GIMP_ITEM_STACK (image->layers);
+
+  return GIMP_LAYER (gimp_item_stack_get_item_by_tattoo (stack, tattoo));
 }
 
 GimpChannel *
 gimp_image_get_channel_by_tattoo (const GimpImage *image,
                                   GimpTattoo       tattoo)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
 
-  return GIMP_CHANNEL (gimp_item_stack_get_item_by_tattoo (GIMP_ITEM_STACK (image->channels),
-                                                           tattoo));
+  stack = GIMP_ITEM_STACK (image->channels);
+
+  return GIMP_CHANNEL (gimp_item_stack_get_item_by_tattoo (stack, tattoo));
 }
 
 GimpVectors *
 gimp_image_get_vectors_by_tattoo (const GimpImage *image,
                                   GimpTattoo       tattoo)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
 
-  return GIMP_VECTORS (gimp_item_stack_get_item_by_tattoo (GIMP_ITEM_STACK (image->vectors),
-                                                           tattoo));
+  stack = GIMP_ITEM_STACK (image->vectors);
+
+  return GIMP_VECTORS (gimp_item_stack_get_item_by_tattoo (stack, tattoo));
 }
 
 GimpLayer *
 gimp_image_get_layer_by_name (const GimpImage *image,
                               const gchar     *name)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
   g_return_val_if_fail (name != NULL, NULL);
 
-  return GIMP_LAYER (gimp_item_stack_get_item_by_name (GIMP_ITEM_STACK (image->layers),
-                                                       name));
+  stack = GIMP_ITEM_STACK (image->layers);
+
+  return GIMP_LAYER (gimp_item_stack_get_item_by_name (stack, name));
 }
 
 GimpChannel *
 gimp_image_get_channel_by_name (const GimpImage *image,
                                 const gchar     *name)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
   g_return_val_if_fail (name != NULL, NULL);
 
-  return GIMP_CHANNEL (gimp_item_stack_get_item_by_name (GIMP_ITEM_STACK (image->channels),
-                                                         name));
+  stack = GIMP_ITEM_STACK (image->channels);
+
+  return GIMP_CHANNEL (gimp_item_stack_get_item_by_name (stack, name));
 }
 
 GimpVectors *
 gimp_image_get_vectors_by_name (const GimpImage *image,
                                 const gchar     *name)
 {
+  GimpItemStack *stack;
+
   g_return_val_if_fail (GIMP_IS_IMAGE (image), NULL);
   g_return_val_if_fail (name != NULL, NULL);
 
-  return GIMP_VECTORS (gimp_item_stack_get_item_by_name (GIMP_ITEM_STACK (image->vectors),
-                                                         name));
+  stack = GIMP_ITEM_STACK (image->vectors);
+
+  return GIMP_VECTORS (gimp_item_stack_get_item_by_name (stack, name));
 }
 
 static GimpItem *



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]