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 167529 - Can't place guides directly over line 1000
Can't place guides directly over line 1000
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Script-Fu
2.2.x
Other Linux
: Normal normal
: 2.2
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-02-15 21:45 UTC by Steffen Wagner
Modified: 2008-01-15 12:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Raises limit to 250000 (725 bytes, patch)
2005-02-16 01:31 UTC, Joao S. O. Bueno
none Details | Review

Description Steffen Wagner 2005-02-15 21:45:21 UTC
Distribution/Version: Fedora 3

It's not possible to place a guide with the menupoint "new guide" (hope that's
the translation, I've got the german Version) in a line or col over 1000. It's
always placed in line 1000, if you enter a value greater 1000.
Comment 1 Sven Neumann 2005-02-15 23:01:04 UTC
Yeah, that spinbutton has an upper limit of 1000. Not really a good idea but it
should be easy to fix that script.
Comment 2 Joao S. O. Bueno 2005-02-16 01:31:23 UTC
Created attachment 37518 [details] [review]
Raises limit to 250000

Will have to be reviewed in a few years again when 600dpi e-paper screens are
out, to edit them 4 TeraPixel digital photos.
Comment 3 Sven Neumann 2005-02-16 10:04:28 UTC
250000 is pretty close to the maximum image dimensions used throughout GIMP. The
value GIMP_MAX_IMAGE_SIZE is defined in libgimpbase/gimplimits.h and is
currently 262144.
Comment 4 weskaggs 2005-02-16 19:43:17 UTC
Okay, committed to both branches:

2005-02-16  Bill Skaggs  <weskaggs@primate.ucdavis.edu>

	* app/plug-ins/script-fu/scripts/guides-new.scm:  committted
	slightly modified patch from  Joao S. O. Bueno Calligaris
	to raise guide position limit to 262144.

Too bad MAX-IMAGE-SIZE is not defined in script-fu land.
Comment 5 Kevin Cozens 2005-02-17 04:28:43 UTC
It is defined now along with MIN-IMAGE-SIZE, MIN-RESOLUTION and MAX-RESOLUTION.