auto levels in krita

[Boudewijn Rempt]

Nice work! You even fixed the coding style by adding the m_ prefix (though I guess it's not needed here:

-        // Update the histogram
+        // Update the m_histogram
)

The two usual places for patches are as attachments to a bug in bugs.kde.org or on git.reviewboard.kde.org, but I'll just build and check and push your patches today.


On Sunday 25 January 2015 Jan 14:47:16 Alex Demko wrote:
> I added a small "auto levels" feature to krita, much like the feature
> in the gimp. I noticed gimp by default presents the R/G/B histograms,
> but by default krita works on the L one. I'm not a color guy, so I'm
> not sure if this makes a big difference. It seems to work nicely,
> though.
> 
> If the team is interested, the patches (1 for the feature, 1 to
> QScopePointer a loose pointer) are here:
> 
> https://demko.ca/~ademko/kritalevels/
> 
> Feed back is appreciated. I tried to match style as much as possible.
> If you 'd prefer I upload my git branch to github (or somewhere else),
> I can do that.
> 
> Thanks
> _______________________________________________
> Krita mailing list
> kimageshop at kde.org
> https://mail.kde.org/mailman/listinfo/kimageshop

-- 
Boudewijn Rempt
http://www.valdyas.org, http://www.krita.org