Removed unnecessary CompareExchange calls in the ThrowOnNextStartupForTests property setter in SampleProfilerSession.cs.

One straggler from the PR review feedback that I somehow missed: explicitly discard an unused return value in one of the tests.

Author: logiclrd

src/Sentry.Profiling/SampleProfilerSession.cs

@@ -61,13 +61,7 @@ private SampleProfilerSession(SentryStopwatch stopwatch, EventPipeSession sessio
     internal static bool ThrowOnNextStartupForTests
     {
         get { return _throwOnNextStartupForTests; }
-        set
-        {
-            if (value)
-                _throwOnNextStartupForTests.CompareExchange(true, false);
-            else
-                _throwOnNextStartupForTests.CompareExchange(false, true);
-        }
+        set { _throwOnNextStartupForTests = value; }
     }
 
     public static SampleProfilerSession StartNew(IDiagnosticLogger? logger = null)

test/Sentry.Tests/Internals/InterlockedBooleanTests.cs

@@ -96,7 +96,7 @@ public void InterlockedBoolean_Exchange_SetsExpectedNewState(bool initialState,
         var expected = ToTBool(newValue);
 
         // Act
-        var result = sut.Exchange(newValue);
+        var _ = sut.Exchange(newValue);
 
         // Assert
         sut.ValueForTests.Should().Be(expected);