[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-kimageshop
Subject: Re: [calligra] krita/plugins/filters/blur: parenthesis to know the
From: Cyrille Berger Skott <cberger () cberger ! net>
Date: 2011-07-01 8:27:32
Message-ID: 201107011027.32515.cberger () cberger ! net
[Download RAW message or body]
Hi,
It does break the test "krita-image-KisAsyncMergerTest" (see
http://my.cdash.org/viewTest.php?onlydelta&buildid=204196). But I am guessing
KisAsyncMergerTest was relying on the buggy behaviour of kis_blur_filter
configuration. Dmitry, can you have a look to adjust the configuration of the
blur filter used in that test ? Or if the test image need to be regenerated.
On Thursday 30 June 2011, Jaime Torres wrote:
> Git commit 3cea838598fd6f148e0cef78a43e28fa7e73f44d by Jaime Torres.
> Committed on 30/06/2011 at 20:12.
> Pushed by jtamate into branch 'master'.
>
> parenthesis to know the exact meaning of an expresion with ? operator.
>
> part of review 101803
>
> M +1 -1 krita/plugins/filters/blur/kis_blur_filter.cpp
>
> http://commits.kde.org/calligra/3cea838598fd6f148e0cef78a43e28fa7e73f44d
>
> diff --git a/krita/plugins/filters/blur/kis_blur_filter.cpp
> b/krita/plugins/filters/blur/kis_blur_filter.cpp index 3a45bc2..9e3833b
> 100644
> --- a/krita/plugins/filters/blur/kis_blur_filter.cpp
> +++ b/krita/plugins/filters/blur/kis_blur_filter.cpp
> @@ -78,7 +78,7 @@ void KisBlurFilter::process(KisPaintDeviceSP device,
> uint halfHeight = (config->getProperty("halfHeight", value)) ?
> value.toUInt() : 5; uint height = 2 * halfHeight + 1;
> int rotate = (config->getProperty("rotate", value)) ? value.toInt() :
> 0; - int strength = 100 - (config->getProperty("strength", value)) ?
> value.toUInt() : 0; + int strength = 100 -
> (config->getProperty("strength", value) ? value.toUInt() : 0);
>
> int hFade = (halfWidth * strength) / 100;
> int vFade = (halfHeight * strength) / 100;
--
Cyrille Berger Skott
_______________________________________________
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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic