From c2f17a10723f29fb8d6adf846e32f5922f309484 Mon Sep 17 00:00:00 2001 From: NinaLua <168073229+NinaLua@users.noreply.github.com> Date: Fri, 7 Mar 2025 16:38:47 +0800 Subject: [PATCH] refactor: using slices.Contains to simplify the code (#3495) Signed-off-by: NinaLua --- pkg/sfu/streamtrackermanager.go | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/pkg/sfu/streamtrackermanager.go b/pkg/sfu/streamtrackermanager.go index a0e35d65a..d341f40c7 100644 --- a/pkg/sfu/streamtrackermanager.go +++ b/pkg/sfu/streamtrackermanager.go @@ -15,6 +15,7 @@ package sfu import ( + "slices" "sort" "sync" "time" @@ -480,7 +481,7 @@ func (s *StreamTrackerManager) getLayeredBitrateLocked() ([]int32, Bitrates) { for i, tracker := range s.trackers { if tracker != nil { tls := make([]int64, buffer.DefaultMaxLayerTemporal+1) - if s.hasSpatialLayerLocked(int32(i)) { + if slices.Contains(s.availableLayers ,int32(i)) { tls = tracker.BitrateTemporalCumulative() } @@ -509,16 +510,6 @@ func (s *StreamTrackerManager) getLayeredBitrateLocked() ([]int32, Bitrates) { return availableLayers, br } -func (s *StreamTrackerManager) hasSpatialLayerLocked(layer int32) bool { - for _, l := range s.availableLayers { - if l == layer { - return true - } - } - - return false -} - func (s *StreamTrackerManager) addAvailableLayer(layer int32) { s.lock.Lock() hasLayer := false