koffice/krita/plugins/paintops

LukasT.dev@gmail.com lukast.dev at gmail.com
Sun Apr 25 14:28:39 CEST 2010


I suppose that this feature is same as density in Soft brush.
I would call it density here too. 

Why don't you make it part of the mask generator code? The feature seems to be 
part of that code. Just my point of view of the design.

Regarding performance:
I would add if statement when the randomness is 0 to avoid computing random 
numbers. Generation of the random numbers is slow.


On Saturday 24 April 2010 11:25:41 Cyrille Berger wrote:
> SVN commit 1118286 by berger:
> 
> add random salt to auto brush masks
> 
> 
>  M  +10 -4     libbrush/kis_auto_brush.cpp
>  M  +1 -1      libbrush/kis_auto_brush.h
>  M  +2 -1      libbrush/kis_auto_brush_factory.cpp
>  M  +2 -2      libbrush/tests/kis_auto_brush_test.cpp
>  M  +5 -1      libpaintop/kis_auto_brush_widget.cpp
>  M  +15 -18    libpaintop/wdgautobrush.ui
> 
 


More information about the kimageshop mailing list