refactor(span): Simplify span error handling in Hub class

Removed the _endActiveSpan method and integrated its functionality directly into the error handling of startSpan. Now, spans are ended and their status is set to error in the catch block, improving code clarity and reducing redundancy.

Author: buenaflor

packages/dart/lib/src/hub.dart

@@ -673,11 +673,13 @@ class Hub {
               () => callback(recordingSpan),
               zoneValues: {_scopeKey: forkedScope},
             );
-            _endActiveSpan(recordingSpan, forkedScope);
             return result;
           } catch (_) {
-            _endActiveSpan(recordingSpan, forkedScope, isError: true);
+            span.status = SentrySpanStatusV2.error;
             rethrow;
+          } finally {
+            span.end();
+            forkedScope.removeActiveSpan(span);
           }
         }();
     }
@@ -714,11 +716,13 @@ class Hub {
             () => callback(recordingSpan),
             zoneValues: {_scopeKey: forkedScope},
           );
-          _endActiveSpan(recordingSpan, forkedScope);
           return result;
         } catch (_) {
-          _endActiveSpan(recordingSpan, forkedScope, isError: true);
+          span.status = SentrySpanStatusV2.error;
           rethrow;
+        } finally {
+          span.end();
+          forkedScope.removeActiveSpan(span);
         }
     }
   }
@@ -728,18 +732,6 @@ class Hub {
     return parentScope.clone()..setActiveSpan(span);
   }
 
-  void _endActiveSpan(
-    RecordingSentrySpanV2 span,
-    Scope forkedScope, {
-    bool isError = false,
-  }) {
-    if (isError) {
-      span.status = SentrySpanStatusV2.error;
-    }
-    span.end();
-    forkedScope.removeActiveSpan(span);
-  }
-
   SentrySpanV2 startInactiveSpan(
     String name, {
     Map<String, SentryAttribute>? attributes,