diff options
author | Ray Molenkamp <github@lazydodo.com> | 2020-02-12 22:49:34 +0300 |
---|---|---|
committer | Ray Molenkamp <github@lazydodo.com> | 2020-02-12 22:49:34 +0300 |
commit | 9fe469c110940af5d2525158305d5d365bd15276 (patch) | |
tree | 3944557e2117fbe67c1ee5bfcd793bc5d3b9a96c /source/blender/collada/MeshImporter.cpp | |
parent | dd9dfadaac9b17d0a5f41545925668717b662b08 (diff) |
Cleanup/MSVC: Enable C++ conformance mode on compiler versions that support it.
MSVC has a conformance mode (/permissive-) where the C++ standard is more strictly
enforced. This mode is available on MSVC 15.5+ [1]
This patch enables this mode on compilers that support it and cleans up the few violations it threw up in the process.
- Mantaflow was using M_PI without requesting them using the _USE_MATH_DEFINES define to opt in to non default behaviour.
- Collada did not include the right header for std::cerr, this seemingly was fixed for other platforms already but put inside a platform guard.
- Ghost had some scoping issues regarding uninitialized variables and goto behaviour
[1] https://docs.microsoft.com/en-us/cpp/build/reference/permissive-standards-conformance
Differential Revision: https://developer.blender.org/D6824
Reviewed By: brecht
Diffstat (limited to 'source/blender/collada/MeshImporter.cpp')
-rw-r--r-- | source/blender/collada/MeshImporter.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/source/blender/collada/MeshImporter.cpp b/source/blender/collada/MeshImporter.cpp index 14c09a95d10..bc6dd4202b1 100644 --- a/source/blender/collada/MeshImporter.cpp +++ b/source/blender/collada/MeshImporter.cpp @@ -19,10 +19,7 @@ */ #include <algorithm> - -#if !defined(WIN32) -# include <iostream> -#endif +#include <iostream> /* COLLADABU_ASSERT, may be able to remove later */ #include "COLLADABUPlatform.h" |