Fix warnings related to Vector3::operator= and remove -Wno-deprecated-copy suppression

# Conflicts:
#	cmake/compiler/clang/settings.cmake
This commit is contained in:
Shauren authored and Ovahlord committed 2020-07-13 20:37:26 +02:00
1 parent cb4e7821e3
commit 78e7687a05
4 files changed
+29 -19

No files matched your search

-6
View File
@@ -14,12 +14,6 @@ if(WITH_WARNINGS)
-Wno-mismatched-tags
-Woverloaded-virtual)
if(CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 10)
target_compile_options(trinity-warning-interface
INTERFACE
-Wno-deprecated-copy) # warning in g3d
endif()
message(STATUS "Clang: All warnings enabled")
endif()
-4
View File
@@ -33,10 +33,6 @@ if( WITH_WARNINGS )
-Wfatal-errors
-Woverloaded-virtual)
target_compile_options(trinity-warning-interface
INTERFACE
-Wno-deprecated-copy) # warning in g3d
message(STATUS "GCC: All warnings enabled")
endif()
+28
View File
@@ -0,0 +1,28 @@
diff --git a/dep/g3dlite/include/G3D/Vector3.h b/dep/g3dlite/include/G3D/Vector3.h
index ef1e39d821..1c42513b45 100644
--- a/dep/g3dlite/include/G3D/Vector3.h
+++ b/dep/g3dlite/include/G3D/Vector3.h
@@ -128,7 +128,7 @@ public:
Axis primaryAxis() const;
// assignment and comparison
- Vector3& __fastcall operator= (const Vector3& rkVector);
+ Vector3& operator=(const Vector3& rkVector) = default;
Vector3& operator=(const Any& a);
bool operator== (const Vector3& rkVector) const;
bool operator!= (const Vector3& rkVector) const;
@@ -627,14 +627,6 @@ inline float& Vector3::operator[] (int i) {
}
-//----------------------------------------------------------------------------
-inline Vector3& Vector3::operator= (const Vector3& rkVector) {
- x = rkVector.x;
- y = rkVector.y;
- z = rkVector.z;
- return *this;
-}
-
//----------------------------------------------------------------------------
inline bool Vector3::fuzzyEq(const Vector3& other) const {
+1 -9
View File
@@ -128,7 +128,7 @@ public:
Axis primaryAxis() const;
// assignment and comparison
Vector3& __fastcall operator= (const Vector3& rkVector);
Vector3& operator=(const Vector3& rkVector) = default;
Vector3& operator=(const Any& a);
bool operator== (const Vector3& rkVector) const;
bool operator!= (const Vector3& rkVector) const;
@@ -627,14 +627,6 @@ inline float& Vector3::operator[] (int i) {
}
//----------------------------------------------------------------------------
inline Vector3& Vector3::operator= (const Vector3& rkVector) {
x = rkVector.x;
y = rkVector.y;
z = rkVector.z;
return *this;
}
//----------------------------------------------------------------------------
inline bool Vector3::fuzzyEq(const Vector3& other) const {