[Okular-devel] Review Request 118950: Use correct #include for abs(int)

Vadim Zhukov persgray at gmail.com
Thu Jul 10 10:12:53 UTC 2014



> On Июль 10, 2014, 12:35 д.п., Kai Uwe Broulik wrote:
> > Same as with Review 119133: What about just using qAbs instead of std::abs?

Good idea. I've updated a patch. Still works here.


- Vadim


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


On Июль 10, 2014, 2:12 п.п., Vadim Zhukov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118950/
> -----------------------------------------------------------
> 
> (Updated Июль 10, 2014, 2:12 п.п.)
> 
> 
> Review request for Okular, Jon Mease, Luigi Toscano, and Tingnan Zhang.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> Both C and C++ have abs() function. C one is abs(int), declared in <stdlib.h>; and C++ has std::abs(int), std::abs(long) and std::abs(long long) in <cstdlib.h>. But C++ has even more overloaded abs() versions in <cmath>, that do operate on floating point values.
> 
> core/utils.cpp incorrectly includes <cmath> while std::abs() calls near line 140 use integer versions really:
> 
>                     && std::abs(static_cast<int>(selectedOutput->edid()->width()*10) - szMM.width()) < 10
>                     && std::abs(static_cast<int>(selectedOutput->edid()->height()*10) - szMM.height()) < 10)
> 
> This patch changes include to <cstdlib>.
> 
> 
> Diffs
> -----
> 
>   core/utils.cpp 71442f0 
> 
> Diff: https://git.reviewboard.kde.org/r/118950/diff/
> 
> 
> Testing
> -------
> 
> Tested on OpenBSD/i386. Without this patch, build fails.
> 
> 
> Thanks,
> 
> Vadim Zhukov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20140710/de40b63a/attachment-0001.html>


More information about the Okular-devel mailing list