From d27187215da1930c43a22c542d18f7578671ed74 Mon Sep 17 00:00:00 2001 From: Will Hawkins Date: Fri, 14 Jul 2023 11:02:56 -0400 Subject: Use defer to unlock probe data point channel lock This approach *seems* like a good idea. Signed-off-by: Will Hawkins --- rpm/rpm.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rpm/rpm.go b/rpm/rpm.go index ff483ff..6e14418 100644 --- a/rpm/rpm.go +++ b/rpm/rpm.go @@ -287,6 +287,7 @@ func ResponsivenessProber[BucketType utilities.Number]( } dataPointsLock.Lock() + defer dataPointsLock.Unlock() // Now we have our four data points (three in the foreign probe data point and one in the self probe data point) if dataPoints != nil { measurement := ResponsivenessProbeResult{ @@ -298,7 +299,6 @@ func ResponsivenessProber[BucketType utilities.Number]( Measure: utilities.Some[ResponsivenessProbeResult](measurement), } } - dataPointsLock.Unlock() }() } if debug.IsDebug(debugging.Level) { -- cgit v1.2.3