force strict ISO C++ (-Wpedantic)

for Fem, Import, PartDesign, Robot, Sketcher, TechDraw
FreeCAD · Sep 18, 2019 · d0d92b0 · d0d92b0
1 parent a247b7e
commit d0d92b0
Show file tree

Hide file tree

Showing 64 changed files with 268 additions and 257 deletions.
diff --git a/src/Mod/Fem/App/CMakeLists.txt b/src/Mod/Fem/App/CMakeLists.txt
@@ -1,7 +1,9 @@
 
 # Many warnings caused by vtk
 if(CMAKE_COMPILER_IS_CLANGXX)
-    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-inconsistent-missing-override")
+    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-inconsistent-missing-override -Wno-pedantic") # needed for vtk headers
+elseif(CMAKE_COMPILER_IS_GNUCXX)
+    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-pedantic") # needed for vtk headers
 endif()
 
 if(MSVC)

diff --git a/src/Mod/Fem/App/FemConstraint.cpp b/src/Mod/Fem/App/FemConstraint.cpp
@@ -72,7 +72,7 @@ double round(double r) {
 }
 #endif
 
-PROPERTY_SOURCE(Fem::Constraint, App::DocumentObject);
+PROPERTY_SOURCE(Fem::Constraint, App::DocumentObject)
 
 Constraint::Constraint()
 {

diff --git a/src/Mod/Fem/App/FemConstraintBearing.cpp b/src/Mod/Fem/App/FemConstraintBearing.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintBearing, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintBearing, Fem::Constraint)
 
 ConstraintBearing::ConstraintBearing()
 {

diff --git a/src/Mod/Fem/App/FemConstraintContact.cpp b/src/Mod/Fem/App/FemConstraintContact.cpp
@@ -37,7 +37,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintContact, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintContact, Fem::Constraint)
 
 ConstraintContact::ConstraintContact()
 {

diff --git a/src/Mod/Fem/App/FemConstraintDisplacement.cpp b/src/Mod/Fem/App/FemConstraintDisplacement.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintDisplacement, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintDisplacement, Fem::Constraint)
 
 ConstraintDisplacement::ConstraintDisplacement()
 {

diff --git a/src/Mod/Fem/App/FemConstraintFixed.cpp b/src/Mod/Fem/App/FemConstraintFixed.cpp
@@ -49,7 +49,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintFixed, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintFixed, Fem::Constraint)
 
 ConstraintFixed::ConstraintFixed()
 {

diff --git a/src/Mod/Fem/App/FemConstraintFluidBoundary.cpp b/src/Mod/Fem/App/FemConstraintFluidBoundary.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintFluidBoundary, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintFluidBoundary, Fem::Constraint)
 
 // also defined in TaskFemConstraintFluidBoundary.cpp and FoamCaseBuilder/BasicBuilder.py, update simultaneously
 // the second (index 1) item is the default enum, as index 0 causes compiling error

diff --git a/src/Mod/Fem/App/FemConstraintForce.cpp b/src/Mod/Fem/App/FemConstraintForce.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintForce, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintForce, Fem::Constraint)
 
 ConstraintForce::ConstraintForce()
 {

diff --git a/src/Mod/Fem/App/FemConstraintGear.cpp b/src/Mod/Fem/App/FemConstraintGear.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintGear, Fem::ConstraintBearing);
+PROPERTY_SOURCE(Fem::ConstraintGear, Fem::ConstraintBearing)
 
 ConstraintGear::ConstraintGear()
 {

diff --git a/src/Mod/Fem/App/FemConstraintHeatflux.cpp b/src/Mod/Fem/App/FemConstraintHeatflux.cpp
@@ -39,7 +39,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintHeatflux, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintHeatflux, Fem::Constraint)
 
 static const char* ConstraintTypes[] = {"DFlux","Convection", NULL};
 

diff --git a/src/Mod/Fem/App/FemConstraintInitialTemperature.cpp b/src/Mod/Fem/App/FemConstraintInitialTemperature.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintInitialTemperature, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintInitialTemperature, Fem::Constraint)
 
 ConstraintInitialTemperature::ConstraintInitialTemperature()
 {

diff --git a/src/Mod/Fem/App/FemConstraintPlaneRotation.cpp b/src/Mod/Fem/App/FemConstraintPlaneRotation.cpp
@@ -37,7 +37,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintPlaneRotation, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintPlaneRotation, Fem::Constraint)
 
 ConstraintPlaneRotation::ConstraintPlaneRotation()
 {

diff --git a/src/Mod/Fem/App/FemConstraintPressure.cpp b/src/Mod/Fem/App/FemConstraintPressure.cpp
@@ -37,7 +37,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintPressure, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintPressure, Fem::Constraint)
 
 ConstraintPressure::ConstraintPressure()
 {

diff --git a/src/Mod/Fem/App/FemConstraintPulley.cpp b/src/Mod/Fem/App/FemConstraintPulley.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintPulley, Fem::ConstraintGear);
+PROPERTY_SOURCE(Fem::ConstraintPulley, Fem::ConstraintGear)
 
 ConstraintPulley::ConstraintPulley()
 {

diff --git a/src/Mod/Fem/App/FemConstraintTemperature.cpp b/src/Mod/Fem/App/FemConstraintTemperature.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintTemperature, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintTemperature, Fem::Constraint)
 
 static const char* ConstraintTypes[] = {"CFlux","Temperature", NULL};
 

diff --git a/src/Mod/Fem/App/FemConstraintTransform.cpp b/src/Mod/Fem/App/FemConstraintTransform.cpp
@@ -37,7 +37,7 @@
 
 using namespace Fem;
 
-PROPERTY_SOURCE(Fem::ConstraintTransform, Fem::Constraint);
+PROPERTY_SOURCE(Fem::ConstraintTransform, Fem::Constraint)
 
 static const char* TransformTypes[] = {"Cylindrical","Rectangular", NULL};
 

diff --git a/src/Mod/Fem/App/FemMeshProperty.cpp b/src/Mod/Fem/App/FemMeshProperty.cpp
@@ -40,7 +40,7 @@
 
 using namespace Fem;
 
-TYPESYSTEM_SOURCE(Fem::PropertyFemMesh , App::PropertyComplexGeoData);
+TYPESYSTEM_SOURCE(Fem::PropertyFemMesh , App::PropertyComplexGeoData)
 
 PropertyFemMesh::PropertyFemMesh() : _FemMesh(new FemMesh)
 {

diff --git a/src/Mod/Fem/App/PropertyPostDataObject.cpp b/src/Mod/Fem/App/PropertyPostDataObject.cpp
@@ -55,7 +55,7 @@
 
 using namespace Fem;
 
-TYPESYSTEM_SOURCE(Fem::PropertyPostDataObject , App::Property);
+TYPESYSTEM_SOURCE(Fem::PropertyPostDataObject , App::Property)
 
 PropertyPostDataObject::PropertyPostDataObject()
 {

diff --git a/src/Mod/Fem/Gui/CMakeLists.txt b/src/Mod/Fem/Gui/CMakeLists.txt
@@ -1,7 +1,9 @@
 
 # Many warnings caused by vtk
 if(CMAKE_COMPILER_IS_CLANGXX)
-    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-inconsistent-missing-override")
+    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-inconsistent-missing-override -Wno-pedantic") # needed for vtk headers
+elseif(CMAKE_COMPILER_IS_GNUCXX)
+    set(CMAKE_CXX_FLAGS " ${CMAKE_CXX_FLAGS} -Wno-pedantic") # needed for vtk headers
 endif()
 
 if(MSVC)