dcaliste updated this revision to Diff 40892. dcaliste added a comment.
Update the patch not to use the form ```qBound(qreal(0.0), a, qreal(1.0))``` but ```qBound<qreal>(0.0, a, 1.0)``` instead. For reference, original form was ```qBound(0.0, a, 1.0)``` which was causing issues when qreal <-> float, because of mixed doubles and floats. REPOSITORY R8 Calligra CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15053?vs=40358&id=40892 REVISION DETAIL https://phabricator.kde.org/D15053 AFFECTED FILES libs/pigment/KoColorConversions.cpp libs/pigment/KoColorSpace.cpp libs/pigment/KoColorSpace.h libs/pigment/KoColorSpaceAbstract.h libs/pigment/KoColorSpaceTraits.h libs/pigment/colorprofiles/KoDummyColorProfile.cpp libs/pigment/colorprofiles/KoDummyColorProfile.h libs/pigment/colorspaces/KoAlphaColorSpace.h libs/pigment/colorspaces/KoLabColorSpace.cpp libs/pigment/colorspaces/KoLabColorSpace.h libs/pigment/colorspaces/KoRgbU16ColorSpace.cpp libs/pigment/colorspaces/KoRgbU16ColorSpace.h libs/pigment/colorspaces/KoRgbU8ColorSpace.cpp libs/pigment/colorspaces/KoRgbU8ColorSpace.h libs/pigment/colorspaces/KoSimpleColorSpace.h To: dcaliste, leinir, danders, #calligra:_3.0 Cc: anthonyfieroni, Calligra-Devel-list, cochise, vandenoever