dmaax/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

LibMedia: Remove an unnecessary lock in the audio sink's state dispatch

Browse source 
Both fields used in the deferred invoke are written by the main thread,
we shouldn't need to worry about a race here.
This commit is contained in:
Zaggy1024 2026-06-09 17:20:36 -05:00 committed by Gregory Bertilson
parent d0edc33597
commit eeeabcaef1
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Libraries/LibMedia/Sinks/AudioPlaybackSink.cpp
View file

@ -390,7 +390,6 @@ void AudioPlaybackSink::OutputThreadData::dispatch_state_if_changed(PipelineStat
return;
m_last_dispatched_status = status;
m_main_thread_event_loop.deferred_invoke([self = NonnullRefPtr(*this), status, seek_id] {
Sync::MutexLocker locker { self->m_output_mutex };
if (self->m_seek_id != seek_id)
return;
if (self->m_on_state_changed)