[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5658 Include <cmath> consistently, not <math.h>

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Description has changed:

Diff:

--- old
+++ new
@@ -1,9 +1,9 @@
 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.
+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.
+`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

---

[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:05 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 address@hidden 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
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto