Review Request 123867: Fix build on OSX clang and replace another abs with std::abs

Allen Winter winter at kde.org
Mon May 25 17:39:32 UTC 2015


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


This is incorrect.  std::abs comes from cstdlib, not cmath
please fix accordingly.  I have it compiling here fine on my macbook with #include <cstdlib> instead of #include <cmath>

- Allen Winter


On May 21, 2015, 8:19 a.m., Bhushan Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123867/
> -----------------------------------------------------------
> 
> (Updated May 21, 2015, 8:19 a.m.)
> 
> 
> Review request for KDE Frameworks, Marko Käning, Milian Wolff, and Scarlett Clark.
> 
> 
> Repository: ki18n
> 
> 
> Description
> -------
> 
> https://build.kde.org/job/ki18n%20master%20kf5-qt5/PLATFORM=OSX,compiler=clang/1/console
> 
> Follow up to,
> 
> commit 8d392e523e4556f9e434f3e02890fcacfcb492e4
> Author: Milian Wolff <mail at milianw.de>
> Date:   Mon May 18 10:52:29 2015 +0200
> 
>     Fix compiler warning about potential truncation.
>     
>     klocalizedstring.cpp:1126:46: warning: absolute value function 'abs'
>     given an argument of type 'long' but has parameter of type 'int'
>     which may cause truncation of value
>     
>             kls.d->number = static_cast<pluraln>(abs(a));
>                                                  ^
>     klocalizedstring.cpp:1126:46: note: use function 'std::abs' instead
> 
> 
> Diffs
> -----
> 
>   src/klocalizedstring.cpp 910f44d 
> 
> Diff: https://git.reviewboard.kde.org/r/123867/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bhushan Shah
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150525/85e8f0ad/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list