- **Patch**: review --> countdown
- **Comment**:
Patch on countdown for Jan 16th
---
** [issues:#5658] Include <cmath> consistently, not <math.h>**
**Status:** Started
**Created:** Thu Jan 09, 2020 09:45 PM UTC by Dan Eble
**Last Updated:** Sun Jan 12, 2020 08:25 PM UTC
**Owner:** Dan Eble
In the past, we had problems with std::isinf and std::isnan not being
provided by `<cmath>` because they are C++11 features, so we were
including `<math.h>` in some places.
Now, we are using C++11. Because the code is also graced with
`using namespace std;` all over the place (see issue 4550), ambiguity
may arise. Using `<cmath>` consistently should improve the situation.
https://codereview.appspot.com/569220043
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.
_______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
