Re: [Gimp-developer] adding option to crop tool

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

 



On Wed, Sep 29, 2004 at 06:19:15PM +0200, Dave Neary wrote:
> 
> Hi,
> 
> Quoting Olivier <lists@xxxxxxxxxxxxxxxxx>:
> <snip>
> >   gboolean         blank_outer_region;
> <snip>
> >   GIMP_CONFIG_INSTALL_PROP_BOOLEAN (object_class, PROP_BLANK_OUTER_REGION,
> >                                     "blank-outer-region", NULL,
> >                                     FALSE,
> >                                     0);
> 
> <snip>
> 
> >       options->blank_outer_region = g_value_get_enum (value);
> >       g_value_set_enum (value, options->blank_outer_region);
> 
> This looks rather suspicions - you're declaring a gboolean as a BOOLEAN
> property, and then using get_ and set_enum to get and set its value. Try
> changing these to get_ and set_boolean and see what happens.

how blind did I become.... thanks for noticing that!

regards,
	Olivier

[Index of Archives]     [Video For Linux]     [Photo]     [Yosemite News]     [gtk]     [GIMP for Windows]     [KDE]     [GEGL]     [Gimp's Home]     [Gimp on GUI]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux