anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2025-01-03 08:28:25 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Make it explicit that we need floating point abs

Browse Source 
The 'using namespace std' at the top of the file makes it quite difficult to
understand which abs is used: double std::abs<double>(double) or
int ::abs(int)? Now it is explicit, and the code does not change in subtle
ways if somebody removes the using statement.
This commit is contained in:
Georg Baum 2015-06-28 11:22:42 +02:00
parent 6c3a6ea9bd
commit b8eb4fffee
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/support/lstrings.cpp
View File

@ -1407,7 +1407,7 @@ std::string formatFPNumber(double x)
os << std::fixed; os << std::fixed;
// Prevent outputs of 23.4200000000000017 but output small numbers // Prevent outputs of 23.4200000000000017 but output small numbers
// with at least 6 significant digits. // with at least 6 significant digits.
double const logarithm = log10(abs(x)); double const logarithm = log10(fabs(x));
os << std::setprecision(max(6 - static_cast<int>(round(logarithm)), 0)) << x; os << std::setprecision(max(6 - static_cast<int>(round(logarithm)), 0)) << x;
string result = os.str(); string result = os.str();
if (result.find('.') != string::npos) { if (result.find('.') != string::npos) {