Histogram White Spike bug in Brightness Contrast Filter dialog
William Steidtmann
billstei at hbci.com
Tue Nov 30 02:33:44 CET 2010
Attached is a patch that eliminates the KisHistogram::KisHistogram()
constructor that does not pass a "bounds" parameter when using a paint
device, and modifies all calls to the new KisHistogram constructor that
does use "bounds". Most of these calls are (temporarily) passing
exactBounds (and so this code does not change behaviour there), with the
exception of the Brightness Contrast filter, which now passes image
bounds, and thus fixes the White Spike bug in the Brightness Contrast
filter dialog. This is very much the same as the Levels Filter
histogram patch (see Commit #1200357).
To recreate the (pre-patch) bug:
1) Create a 10x10 image, 8 bit sRGB.
2) Set the color to a dark gray 92,92,92.
3) Use a pixel brush to paint all (most) of the image this dark gray.
4) Use menu Filter->Adjust->Brightness/Contrast curve...
5) Note that the graph has little or no indication (count) in the second
quadrant where this dark gray should be.
After the patch, the second quadrant in step 5) will have an appropriate
pixel count.
William Steidtmann
-------------- next part --------------
A non-text attachment was scrubbed...
Name: BrightnessContrastPatch1.patch
Type: text/x-patch
Size: 6727 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kimageshop/attachments/20101129/bb7b27ca/attachment.patch
More information about the kimageshop
mailing list