fix(ci): enable clang-tidy bugprone-unhandled-self-assignment / cert-oop54-cpp (#26767)

Signed-off-by: kuralme <kuralme@protonmail.com>

.clang-tidy

@@ -141,8 +141,6 @@ Checks:          '*,
                   -cppcoreguidelines-avoid-goto,
                   -hicpp-avoid-goto,
                   -bugprone-branch-clone,
-                  -bugprone-unhandled-self-assignment,
-                  -cert-oop54-cpp,
                   -performance-enum-size,
                   -readability-avoid-nested-conditional-operator,
                   -cppcoreguidelines-prefer-member-initializer,

platforms/common/uORB/Subscription.hpp

@@ -92,6 +92,11 @@ public:
 	// copy assignment
 	Subscription &operator=(const Subscription &other)
 	{
+		// Check for self-assignment
+		if (this == &other) {
+			return *this;
+		}
+
 		unsubscribe();
 		_orb_id = other._orb_id;
 		_instance = other._instance;

src/lib/lat_lon_alt/lat_lon_alt.hpp

@@ -87,6 +87,11 @@ public:
 
 	void operator=(const LatLonAlt &lla)
 	{
+		// Protect against self-assignment
+		if (this == &lla) {
+			return;
+		}
+
 		_latitude_rad = lla.latitude_rad();
 		_longitude_rad = lla.longitude_rad();
 		_altitude = lla.altitude();

src/lib/matrix/matrix/Slice.hpp

@@ -59,6 +59,11 @@ public:
 	// Separate function needed otherwise the default copy constructor matches before the deep copy implementation
 	Self &operator=(const Self &other)
 	{
+		// Protect against self-assignment
+		if (this == &other) {
+			return *this;
+		}
+
 		return this->operator=<M, N>(other);
 	}
 

src/modules/ekf2/EKF/bias_estimator/position_bias_estimator.hpp

@@ -62,7 +62,7 @@ public:
 		}
 	}
 
-	void fuseBias(Vector2f bias, Vector2f bias_var)
+	void fuseBias(const Vector2f &bias, const Vector2f &bias_var)
 	{
 		if ((_sensor_ref != _sensor) && _is_sensor_fusion_active) {
 			_bias[0].fuseBias(bias(0), bias_var(0));