Re: [Gegl-developer] Small thing

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

 



On Thu, Feb 27, 2003 at 07:22:01PM -0800, Daniel Rogers wrote:
> Here is a small memory leak I noticed while poking around in gegl-image.c
> 

I think its right in CVS currently. Color models 
are singletons, they dont need to be ref-ed and 
unref-ed.


> --- gegl-mod/gegl/gegl-image.c	2003-02-27 19:10:31.000000000 -0800
> +++ gegl-orig/gegl/gegl-image.c	2003-02-11 17:39:10.000000000 -0800
> @@ -162,14 +162,6 @@ gegl_image_set_color_model (GeglImage * 
>    g_return_if_fail (self != NULL);
>    g_return_if_fail (GEGL_IS_IMAGE (self));
>  
> -  if (color_model)
> -    {
> -      g_object_ref(color_model);
> -    }
> -  if (self->color_model)
> -    {
> -      g_object_unref(color_model);
> -    }
>    self->color_model = color_model;
>  }
>  


[Index of Archives]     [Yosemite News]     [Yosemite Photos]     [gtk]     [GIMP Users]     [KDE]     [Gimp's Home]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux