Apply lint changes etc.

ComputationalBiomechanicsLab · Feb 27, 2024 · 727054a · 727054a
1 parent 63c9658
commit 727054a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 17 deletions.
diff --git a/src/oscar/Graphics/GraphicsImplementation.cpp b/src/oscar/Graphics/GraphicsImplementation.cpp
@@ -4647,7 +4647,7 @@ namespace
             }
 
             auto copy = format();
-            copy.insert(std::move(desc));
+            copy.insert(desc);
             setFormat(copy);
             return true;
         }

diff --git a/src/oscar/Maths/EulerPerspectiveCamera.h b/src/oscar/Maths/EulerPerspectiveCamera.h
@@ -17,11 +17,11 @@ namespace osc
         Mat4 getViewMtx() const;
         Mat4 getProjMtx(float aspectRatio) const;
 
-        Vec3 origin;
-        Radians pitch;
-        Radians yaw;
-        Radians verticalFOV;
-        float znear;
-        float zfar;
+        Vec3 origin = {};
+        Radians pitch = Degrees{0};
+        Radians yaw = Degrees{180};
+        Radians verticalFOV = Degrees{35};
+        float znear = 0.1f;
+        float zfar = 1000.0f;
     };
 }
diff --git a/src/oscar/Maths/MathsImplementation.cpp b/src/oscar/Maths/MathsImplementation.cpp
@@ -462,16 +462,6 @@ std::ostream& osc::operator<<(std::ostream& o, Disc const& d)
 
 // `EulerPerspectiveCamera` implementation
 
-osc::EulerPerspectiveCamera::EulerPerspectiveCamera() :
-    origin{},
-    pitch{},
-    yaw{-180_deg},
-    verticalFOV{35_deg},
-    znear{0.1f},
-    zfar{1000.0f}
-{
-}
-
 Vec3 osc::EulerPerspectiveCamera::getFront() const
 {
     return normalize(Vec3
@@ -691,13 +681,15 @@ void osc::FocusAlongAxis(PolarPerspectiveCamera& camera, size_t axis, bool negat
         case 0: FocusAlongMinusX(camera); break;
         case 1: FocusAlongMinusY(camera); break;
         case 2: FocusAlongMinusZ(camera); break;
+        default: break;
         }
     }
     else {
         switch (axis) {
         case 0: FocusAlongX(camera); break;
         case 1: FocusAlongY(camera); break;
         case 2: FocusAlongZ(camera); break;
+        default: break;
         }
     }
 }