dcaliste created this revision.
dcaliste added reviewers: leinir, danders, Calligra: 3.0.
dcaliste added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
dcaliste requested review of this revision.

REVISION SUMMARY
  On my ARM environment, qreal is interpreted as float, not double. Currently 
in libs/pigment, qreal is implicitly treated like a double, while writing 
something like qmax(v, 0.), where v is a qreal. 0. is a double and the compiler 
complains because there is no function using (float, double).
  
  There are also many conversions that are done between floats and doubles in 
KoColorSpace.cpp. So I've updated all necessary functions to accept qreal 
instead of double or float. As a result, the API has been changed. I've checked 
that it's still compiling on amd64 in addition to arm for instance but I'm not 
sure that I didn't break something in the plugins for instance.
  
  I would like to hear your review on this patch. If I'm creating some issues, 
I'm eager to try to solve them.

REPOSITORY
  R8 Calligra

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: Calligra-Devel-list, cochise, vandenoever

Reply via email to