After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 321692 - plugin filters->distort->curve_bend fails when calling depricated gimp-rotate
plugin filters->distort->curve_bend fails when calling depricated gimp-rotate
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
2.3.x
Other Linux
: Normal normal
: ---
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-11-17 07:00 UTC by Tim Jedlicka
Modified: 2008-01-15 14:05 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Tim Jedlicka 2005-11-17 07:00:31 UTC
Open an image. Select Filters->Distorts->Curve_bend. Check "Automatic preview".
Enter a Rotate value. The following error will popup:
GIMP Message
WARNING: Plug-In "curve_bend"
(/usr/local/gimp-2.3.5/lib/gimp/2.0/plug-ins/curve_bend)
called deprecated procedure 'gimp-rotate'.
It should call 'gimp-drawable-transform-rotate-default' instead!

I tried to replace gimp-rotate in curve_bend.c with the new procedure, but it
then complained about argument 2 being GIMP_PDB_INT32 but expecting
GIMP_PDB_FLOAT. Tried to correct that and it complained about arg #3 - I got
lost after that so could not figure out the fix - sorry.
Comment 1 Manish Singh 2005-11-17 07:28:31 UTC
Thanks for noticing this. Fixed in CVS:

2005-11-16  Manish Singh  <yosh@gimp.org>

        * plug-ins/common/curve_bend.c (p_gimp_rotate): use
        gimp_drawable_transform_rotate_default instead of deprecated
        gimp-rotate PDB call. Fixes bug #321692.
Comment 2 Sven Neumann 2005-11-17 11:15:16 UTC
The plug-in didn't fail though, it only spit out this warning and worked
correctly, did it?
Comment 3 Tim Jedlicka 2005-11-18 06:28:14 UTC
Correct. It did actually do the rotate (although I didn't realize that until you
pointed it out - despite that it said "Warning" - not "Error" or whatever).