diff options
author | Chris Robinson <[email protected]> | 2020-04-20 04:30:14 -0700 |
---|---|---|
committer | Chris Robinson <[email protected]> | 2020-04-20 04:30:14 -0700 |
commit | 947826af1c14e511dc298bbeba0a7dfeedc985a0 (patch) | |
tree | 9813392aa9eefabddf27db2158f264175f617efe | |
parent | b6924407253683493b1361e081475938ad4b22e6 (diff) |
Get rid of an unnecessary variable
-rw-r--r-- | alc/alc.cpp | 34 |
1 files changed, 14 insertions, 20 deletions
diff --git a/alc/alc.cpp b/alc/alc.cpp index d0d8510b..291bb014 100644 --- a/alc/alc.cpp +++ b/alc/alc.cpp @@ -1760,7 +1760,6 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) HrtfRequestMode hrtf_userreq{Hrtf_Default}; HrtfRequestMode hrtf_appreq{Hrtf_Default}; ALCenum gainLimiter{device->LimiterState}; - const ALCuint old_sends{device->NumAuxSends}; ALCuint new_sends{device->NumAuxSends}; DevFmtChannels oldChans; DevFmtType oldType; @@ -1786,7 +1785,7 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) ALuint numMono{device->NumMonoSources}; ALuint numStereo{device->NumStereoSources}; - ALuint numSends{old_sends}; + ALuint numSends{device->NumAuxSends}; #define TRACE_ATTR(a, v) TRACE("%s = %d\n", #a, v) while(attrList[attrIdx]) @@ -2256,6 +2255,7 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) } slotlock.unlock(); + const ALuint num_sends{device->NumAuxSends}; std::unique_lock<std::mutex> srclock{context->mSourceLock}; for(auto &sublist : context->mSourceList) { @@ -2277,7 +2277,7 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) send.GainLF = 1.0f; send.LFReference = HIGHPASSFREQREF; }; - auto send_begin = source->Send.begin()+static_cast<ptrdiff_t>(device->NumAuxSends); + auto send_begin = source->Send.begin() + static_cast<ptrdiff_t>(num_sends); std::for_each(send_begin, source->Send.end(), clear_send); source->PropsClean.clear(std::memory_order_release); @@ -2297,25 +2297,19 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) } auto voicelist = context->getVoicesSpan(); - if(device->NumAuxSends < old_sends) + for(Voice *voice : voicelist) { - const ALuint num_sends{device->NumAuxSends}; /* Clear extraneous property set sends. */ - for(Voice *voice : voicelist) - { - std::fill(std::begin(voice->mProps.Send)+num_sends, std::end(voice->mProps.Send), - VoiceProps::SendData{}); + std::fill(std::begin(voice->mProps.Send)+num_sends, std::end(voice->mProps.Send), + VoiceProps::SendData{}); - std::fill(voice->mSend.begin()+num_sends, voice->mSend.end(), Voice::TargetData{}); - for(auto &chandata : voice->mChans) - { - std::fill(chandata.mWetParams.begin()+num_sends, chandata.mWetParams.end(), - SendParams{}); - } + std::fill(voice->mSend.begin()+num_sends, voice->mSend.end(), Voice::TargetData{}); + for(auto &chandata : voice->mChans) + { + std::fill(chandata.mWetParams.begin()+num_sends, chandata.mWetParams.end(), + SendParams{}); } - } - for(Voice *voice : voicelist) - { + delete voice->mUpdate.exchange(nullptr, std::memory_order_acq_rel); /* Force the voice to stopped if it was stopping. */ @@ -2344,7 +2338,7 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) chandata.mAmbiScale = scales[*(OrderFromChan++)]; chandata.mAmbiSplitter = splitter; chandata.mDryParams = DirectParams{}; - std::fill_n(chandata.mWetParams.begin(), device->NumAuxSends, SendParams{}); + std::fill_n(chandata.mWetParams.begin(), num_sends, SendParams{}); } voice->mFlags |= VOICE_IS_AMBISONIC; @@ -2356,7 +2350,7 @@ static ALCenum UpdateDeviceParams(ALCdevice *device, const ALCint *attrList) { chandata.mPrevSamples.fill(0.0f); chandata.mDryParams = DirectParams{}; - std::fill_n(chandata.mWetParams.begin(), device->NumAuxSends, SendParams{}); + std::fill_n(chandata.mWetParams.begin(), num_sends, SendParams{}); } voice->mFlags &= ~VOICE_IS_AMBISONIC; |