diff options
author | Campbell Barton <ideasman42@gmail.com> | 2019-04-23 04:21:22 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2019-04-23 04:22:25 +0300 |
commit | 3302fbaeb1d5f58d926c06b6153ec5d6f1a9b0be (patch) | |
tree | 5be399e227415fef745bcdb2b7f20a578628718b /source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp | |
parent | 64b4b719ebd5201d27aa25d7fa2d765eabded9b0 (diff) |
Cleanup: style, use braces for compositor
Diffstat (limited to 'source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp')
-rw-r--r-- | source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp | 30 |
1 files changed, 20 insertions, 10 deletions
diff --git a/source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp b/source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp index e375b2fe45b..8d11328b63b 100644 --- a/source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp +++ b/source/blender/compositor/operations/COM_FastGaussianBlurOperation.cpp @@ -89,17 +89,20 @@ void *FastGaussianBlurOperation::initializeTileData(rcti *rect) this->m_sy = this->m_data.sizey * this->m_size / 2.0f; if ((this->m_sx == this->m_sy) && (this->m_sx > 0.0f)) { - for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) + for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) { IIR_gauss(copy, this->m_sx, c, 3); + } } else { if (this->m_sx > 0.0f) { - for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) + for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) { IIR_gauss(copy, this->m_sx, c, 1); + } } if (this->m_sy > 0.0f) { - for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) + for (c = 0; c < COM_NUM_CHANNELS_COLOR; ++c) { IIR_gauss(copy, this->m_sy, c, 2); + } } } this->m_iirgaus = copy; @@ -123,27 +126,34 @@ void FastGaussianBlurOperation::IIR_gauss(MemoryBuffer *src, const unsigned int num_channels = src->get_num_channels(); // <0.5 not valid, though can have a possibly useful sort of sharpening effect - if (sigma < 0.5f) + if (sigma < 0.5f) { return; + } - if ((xy < 1) || (xy > 3)) + if ((xy < 1) || (xy > 3)) { xy = 3; + } // XXX The YVV macro defined below explicitly expects sources of at least 3x3 pixels, // so just skipping blur along faulty direction if src's def is below that limit! - if (src_width < 3) + if (src_width < 3) { xy &= ~1; - if (src_height < 3) + } + if (src_height < 3) { xy &= ~2; - if (xy < 1) + } + if (xy < 1) { return; + } // see "Recursive Gabor Filtering" by Young/VanVliet // all factors here in double.prec. Required, because for single.prec it seems to blow up if sigma > ~200 - if (sigma >= 3.556f) + if (sigma >= 3.556f) { q = 0.9804f * (sigma - 3.556f) + 2.5091f; - else // sigma >= 0.5 + } + else { // sigma >= 0.5 q = (0.0561f * sigma + 0.5784f) * sigma - 0.2568f; + } q2 = q * q; sc = (1.1668 + q) * (3.203729649 + (2.21566 + q) * q); // no gabor filtering here, so no complex multiplies, just the regular coefs. |