Re: A plan for GimpRectangleTool

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

 



Simon Budig skrev:
> Martin Nordholts (enselic@xxxxxxxxx) wrote:
>> With the introduction of helper functions like
>>
>> void gimp_rectangle_tool_constrain_aspect (GimpRectangleTool
>> *rectangle_tool, gdouble aspect, const gchar *apply_on_x, const gchar
>> *apply_on_y)
>> {
>>   /* apply_on_x may be "x1", "x2" or "center_x", same for on_y */
>> }
>
> Hmm, wouldn't an enum for apply_on_* make more sense?
>
> Bye,
>         Simon


Well, that will depend on if it will be used similarly to the
gimp_rectangle_tool_set_other_side-family of functions that were in the
patch I linked to, or, perhaps more likely, if it will be used on the
local x1, x2, etc gint:s in gimp_rectangle_tool_motion. I'd prefer the
former but we'll see how the logic in _motion folds out.

- Martin
_______________________________________________
Gimp-developer mailing list
Gimp-developer@xxxxxxxxxxxxxxxxxxxxxx
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer

[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