aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--al/buffer.cpp2
-rw-r--r--al/eax_exception.cpp4
-rw-r--r--al/error.cpp2
-rw-r--r--alc/alc.cpp42
-rw-r--r--alc/alu.cpp10
-rw-r--r--alc/backends/base.cpp4
-rw-r--r--common/albit.h8
7 files changed, 33 insertions, 39 deletions
diff --git a/al/buffer.cpp b/al/buffer.cpp
index 8a5bee25..3d9caf54 100644
--- a/al/buffer.cpp
+++ b/al/buffer.cpp
@@ -761,7 +761,7 @@ void PrepareCallback(ALCcontext *context, ALbuffer *ALBuf, ALsizei freq,
const ALuint ambiorder{IsBFormat(*DstChannels) ? ALBuf->UnpackAmbiOrder :
(IsUHJ(*DstChannels) ? 1 : 0)};
- constexpr uint line_size{BufferLineSize + MaxPostVoiceLoad};
+ static constexpr uint line_size{BufferLineSize + MaxPostVoiceLoad};
al::vector<al::byte,16>(FrameSizeFromFmt(*DstChannels, *DstType, ambiorder) *
size_t{line_size}).swap(ALBuf->mData);
diff --git a/al/eax_exception.cpp b/al/eax_exception.cpp
index c8ecf79d..fdeecaaa 100644
--- a/al/eax_exception.cpp
+++ b/al/eax_exception.cpp
@@ -30,10 +30,10 @@ std::string EaxException::make_message(
return std::string{};
}
- constexpr auto left_prefix = "[";
+ static constexpr char left_prefix[] = "[";
const auto left_prefix_size = std::string::traits_type::length(left_prefix);
- constexpr auto right_prefix = "] ";
+ static constexpr char right_prefix[] = "] ";
const auto right_prefix_size = std::string::traits_type::length(right_prefix);
const auto what_size =
diff --git a/al/error.cpp b/al/error.cpp
index 8cabf8c3..90671011 100644
--- a/al/error.cpp
+++ b/al/error.cpp
@@ -87,7 +87,7 @@ START_API_FUNC
ContextRef context{GetContextRef()};
if(unlikely(!context))
{
- constexpr ALenum deferror{AL_INVALID_OPERATION};
+ static constexpr ALenum deferror{AL_INVALID_OPERATION};
WARN("Querying error state on null context (implicitly 0x%04x)\n", deferror);
if(TrapALError)
{
diff --git a/alc/alc.cpp b/alc/alc.cpp
index 4c8d0a6c..90da2614 100644
--- a/alc/alc.cpp
+++ b/alc/alc.cpp
@@ -1268,23 +1268,17 @@ void alc_initconfig(void)
#ifdef ALSOFT_EAX
{
- constexpr auto eax_block_name = "eax";
+ static constexpr char eax_block_name[] = "eax";
- const auto eax_enable_opt = ConfigValueBool(nullptr, eax_block_name, "enable");
-
- if (eax_enable_opt)
+ if(const auto eax_enable_opt = ConfigValueBool(nullptr, eax_block_name, "enable"))
{
eax_g_is_enabled = *eax_enable_opt;
-
- if (!eax_g_is_enabled)
- {
+ if(!eax_g_is_enabled)
TRACE("%s\n", "EAX disabled by a configuration.");
- }
}
else
- {
eax_g_is_enabled = true;
- }
+
if(eax_g_is_enabled && DisabledEffects[EAXREVERB_EFFECT])
{
eax_g_is_enabled = false;
@@ -1530,19 +1524,19 @@ void alcSetError(ALCdevice *device, ALCenum errorCode)
std::unique_ptr<Compressor> CreateDeviceLimiter(const ALCdevice *device, const float threshold)
{
- constexpr bool AutoKnee{true};
- constexpr bool AutoAttack{true};
- constexpr bool AutoRelease{true};
- constexpr bool AutoPostGain{true};
- constexpr bool AutoDeclip{true};
- constexpr float LookAheadTime{0.001f};
- constexpr float HoldTime{0.002f};
- constexpr float PreGainDb{0.0f};
- constexpr float PostGainDb{0.0f};
- constexpr float Ratio{std::numeric_limits<float>::infinity()};
- constexpr float KneeDb{0.0f};
- constexpr float AttackTime{0.02f};
- constexpr float ReleaseTime{0.2f};
+ static constexpr bool AutoKnee{true};
+ static constexpr bool AutoAttack{true};
+ static constexpr bool AutoRelease{true};
+ static constexpr bool AutoPostGain{true};
+ static constexpr bool AutoDeclip{true};
+ static constexpr float LookAheadTime{0.001f};
+ static constexpr float HoldTime{0.002f};
+ static constexpr float PreGainDb{0.0f};
+ static constexpr float PostGainDb{0.0f};
+ static constexpr float Ratio{std::numeric_limits<float>::infinity()};
+ static constexpr float KneeDb{0.0f};
+ static constexpr float AttackTime{0.02f};
+ static constexpr float ReleaseTime{0.2f};
return Compressor::Create(device->RealOut.Buffer.size(), static_cast<float>(device->Frequency),
AutoKnee, AutoAttack, AutoRelease, AutoPostGain, AutoDeclip, LookAheadTime, HoldTime,
@@ -2503,7 +2497,7 @@ static size_t GetIntegerv(ALCdevice *device, ALCenum param, const al::span<int>
std::lock_guard<std::mutex> _{device->StateLock};
if(device->Type == DeviceType::Capture)
{
- constexpr int MaxCaptureAttributes{9};
+ static constexpr int MaxCaptureAttributes{9};
switch(param)
{
case ALC_ATTRIBUTES_SIZE:
diff --git a/alc/alu.cpp b/alc/alu.cpp
index 3e35d735..a263960c 100644
--- a/alc/alu.cpp
+++ b/alc/alu.cpp
@@ -504,8 +504,8 @@ inline float ScaleAzimuthFront(float azimuth, float scale)
/* Wraps the given value in radians to stay between [-pi,+pi] */
inline float WrapRadians(float r)
{
- constexpr float Pi{al::numbers::pi_v<float>};
- constexpr float Pi2{Pi*2.0f};
+ static constexpr float Pi{al::numbers::pi_v<float>};
+ static constexpr float Pi2{Pi*2.0f};
if(r > Pi) return std::fmod(Pi+r, Pi2) - Pi;
if(r < -Pi) return Pi - std::fmod(Pi-r, Pi2);
return r;
@@ -1103,7 +1103,7 @@ void CalcPanningAndFilters(Voice *voice, const float xpos, const float ypos, con
/* If the source distance is 0, simulate a plane-wave by using
* infinite distance, which results in a w0 of 0.
*/
- constexpr float w0{0.0f};
+ static constexpr float w0{0.0f};
for(size_t c{0};c < num_channels;c++)
voice->mChans[c].mDryParams.NFCtrlFilter.adjust(w0);
@@ -1265,7 +1265,7 @@ void CalcAttnSourceParams(Voice *voice, const VoiceProps *props, const ContextBa
* decay distance (so it doesn't take any longer to decay
* than the air would allow).
*/
- constexpr float log10_decaygain{-3.0f/*std::log10(ReverbDecayGain)*/};
+ static constexpr float log10_decaygain{-3.0f/*std::log10(ReverbDecayGain)*/};
const float absorb_dist{log10_decaygain / std::log10(airAbsorption)};
DecayDistance[i].HF = minf(absorb_dist, DecayDistance[i].HF);
}
@@ -1383,7 +1383,7 @@ void CalcAttnSourceParams(Voice *voice, const VoiceProps *props, const ContextBa
/* Calculate directional soundcones */
if(directional && props->InnerAngle < 360.0f)
{
- constexpr float Rad2Deg{static_cast<float>(180.0 / al::numbers::pi)};
+ static constexpr float Rad2Deg{static_cast<float>(180.0 / al::numbers::pi)};
const float Angle{Rad2Deg*2.0f * std::acos(-Direction.dot_product(ToSource)) * ConeScale};
float ConeGain, ConeHF;
diff --git a/alc/backends/base.cpp b/alc/backends/base.cpp
index dea29a80..cd1b76ba 100644
--- a/alc/backends/base.cpp
+++ b/alc/backends/base.cpp
@@ -142,9 +142,9 @@ void BackendBase::setDefaultChannelOrder()
#ifdef _WIN32
void BackendBase::setChannelOrderFromWFXMask(uint chanmask)
{
- constexpr uint x51{SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT | SPEAKER_FRONT_CENTER
+ static constexpr uint x51{SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT | SPEAKER_FRONT_CENTER
| SPEAKER_LOW_FREQUENCY | SPEAKER_SIDE_LEFT | SPEAKER_SIDE_RIGHT};
- constexpr uint x51rear{SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT | SPEAKER_FRONT_CENTER
+ static constexpr uint x51rear{SPEAKER_FRONT_LEFT | SPEAKER_FRONT_RIGHT | SPEAKER_FRONT_CENTER
| SPEAKER_LOW_FREQUENCY | SPEAKER_BACK_LEFT | SPEAKER_BACK_RIGHT};
/* Swap a 5.1 mask using the back channels for one with the sides. */
if(chanmask == x51rear) chanmask = x51;
diff --git a/common/albit.h b/common/albit.h
index ad596208..047caab6 100644
--- a/common/albit.h
+++ b/common/albit.h
@@ -99,10 +99,10 @@ constexpr std::enable_if_t<std::is_integral<T>::value && std::is_unsigned<T>::va
int> popcount(T val) noexcept
{
using fast_type = typename detail_::fast_utype<T>::type;
- constexpr fast_type b01010101{detail_::repbits<fast_type>(0x55)};
- constexpr fast_type b00110011{detail_::repbits<fast_type>(0x33)};
- constexpr fast_type b00001111{detail_::repbits<fast_type>(0x0f)};
- constexpr fast_type b00000001{detail_::repbits<fast_type>(0x01)};
+ static constexpr fast_type b01010101{detail_::repbits<fast_type>(0x55)};
+ static constexpr fast_type b00110011{detail_::repbits<fast_type>(0x33)};
+ static constexpr fast_type b00001111{detail_::repbits<fast_type>(0x0f)};
+ static constexpr fast_type b00000001{detail_::repbits<fast_type>(0x01)};
fast_type v{fast_type{val} - ((fast_type{val} >> 1) & b01010101)};
v = (v & b00110011) + ((v >> 2) & b00110011);