Question about ctgmath and tgmath.h

[thomthom]

Sketchup-API-C-Wrapper/SUAPI-CppWrapper/Transformation.cpp

Lines 29 to 33 in 498c19b

	#ifdef WIN32
	#include <ctgmath>
	#else
	#include <tgmath.h>
	#endif // WIN32

Why this ifdef?

I usually only use <cmath>. Looking at <ctgmath> it appear to include <cmath> and `.

Why the need for different includes? Isn't <cmath> enough for this file?

[TommyKaneko]

Would love a Windows developer to assess this....

[thomthom]

When I look up <tgmath.h> I find references to The Open Group Library. Doesn't seem to be part of the standard libs.

I'd recommend sticking with std libs for as much as possible. I think <cmath is enough for the whole file - and that should be platform compatible since it's std library. (Personally I prefer to use the std:: prefixed versions like std::abs over the global non-namespaced abs, fabs C style variants.

[TommyKaneko]

fixed in 2b7e168