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 161573 - Newsprint line spot function malfunctionining
Newsprint line spot function malfunctionining
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
git master
Other All
: Normal minor
: 2.2
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2004-12-17 20:51 UTC by Joao S. O. Bueno
Modified: 2005-06-21 22:13 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Joao S. O. Bueno 2004-12-17 20:51:25 UTC
Create a new image,  
fill it with  white to black blend so that Newsprint has something to do. 
 
Run filters->distorts->newsprint 
Choose "line" as the spot function for any color. 
it all goes fine up to here. 
change the cell size (bigger values make it easier to spot the malfunctioning) 
The problem can be seen in the preview now. If the filter is applied it spoils 
the image.  
 
However, if you apply undo, and rerun the filter with the same parameters it 
works fine. 
 
I do not remember this bug earlier - I suppose it has been the ading of the 
preview that triggered it.
Comment 1 weskaggs 2004-12-17 22:13:07 UTC
I can reproduce this.
Comment 2 Michael Natterer 2005-01-15 19:56:35 UTC
I can see absolutely no difference in the output after running
interactively and with last_vals.
Comment 3 weskaggs 2005-01-16 18:39:36 UTC
More detailed recipe:
1) Start GIMP.
2) Create new image in default size.
3) Fill image with black-white gradient from left to right.
4) Start "Newsprint"
5) Change spot function for Red from Round to Line.
6) Increase Cell size from 10 to 15.  (You now see the problem.)
7) Press OK.
8) Undo
9) Do "Repeat Newsprint".  (You now see what it ought to look like.)

The key thing is changing the cell size.
Comment 4 weskaggs 2005-06-21 22:13:47 UTC
Turns out that the code included some "optimizations" that went badly wrong when
the newsprint() function is executed more than once (as in previewing).  Fixed
in both branches.

2005-06-21  Bill Skaggs  <weskaggs@primate.ucdavis.edu>

	* plug-ins/common/newsprint.c: remove "optimization" that interacts
	badly with preview; fixes bug #161573.