D15053: Consistently use qreal in libs/pigment

Damien Caliste noreply at phabricator.kde.org
Mon Sep 3 09:14:32 BST 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20180903/9f828511/attachment.htm>


More information about the calligra-devel mailing list