[Okular-devel] Review Request 119133: Include <cstdlib> instead of <cmath> for std::abs(int).

Raphael Kubo da Costa rakuco at FreeBSD.org
Mon Jul 14 22:33:16 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119133/
-----------------------------------------------------------

(Updated jul. 14, 2014, 10:33 p.m.)


Status
------

This change has been discarded.


Review request for Okular.


Repository: okular


Description
-------

This fixes the build with libc++ after 8872524:

core/utils.cpp:142:24: error: call to 'abs' is ambiguous
                    && std::abs(static_cast<int>(selectedOutput->edid()->width()*10) - szMM.width()) < 10
                       ^~~~~~~~
/usr/include/c++/v1/cmath:660:1: note: candidate function
abs(float __x) _NOEXCEPT {return fabsf(__x);}
^
/usr/include/c++/v1/cmath:664:1: note: candidate function
abs(double __x) _NOEXCEPT {return fabs(__x);}
^
/usr/include/c++/v1/cmath:668:1: note: candidate function
abs(long double __x) _NOEXCEPT {return fabsl(__x);}
^

core/utils.cpp:143:24: error: call to 'abs' is ambiguous
                    && std::abs(static_cast<int>(selectedOutput->edid()->height()*10) - szMM.height()) < 10)
                       ^~~~~~~~
/usr/include/c++/v1/cmath:660:1: note: candidate function
abs(float __x) _NOEXCEPT {return fabsf(__x);}
^
/usr/include/c++/v1/cmath:664:1: note: candidate function
abs(double __x) _NOEXCEPT {return fabs(__x);}
^
/usr/include/c++/v1/cmath:668:1: note: candidate function
abs(long double __x) _NOEXCEPT {return fabsl(__x);}
^


Diffs
-----

  core/utils.cpp 71442f0e60bc3998c676ac9670964dbd9213e146 

Diff: https://git.reviewboard.kde.org/r/119133/diff/


Testing
-------


Thanks,

Raphael Kubo da Costa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20140714/9afee938/attachment.html>


More information about the Okular-devel mailing list