[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-kimageshop
Subject:    Re: koffice/krita/plugins/paintops
From:       "LukasT.dev () gmail ! com" <lukast ! dev () gmail ! com>
Date:       2010-04-25 12:28:39
Message-ID: 201004251428.39749.LukasT.dev () gmail ! com
[Download RAW message or body]

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
> 
 
_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop
[prev in list] [next in list] [prev in thread] [next in thread]