Skip to content

Commit ae9f0e6

Browse files
committed
More types and casting changes
1 parent 9a79b18 commit ae9f0e6

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

tests/FileWritingTests.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ template <typename T>
1414
void writeTestAudioFile (int numChannels, int sampleRate, int bitDepth, AudioFileFormat format)
1515
{
1616
std::string sampleType;
17-
float sampleRateAsFloat = (float) sampleRate;
17+
float sampleRateAsFloat = static_cast<float> (sampleRate);
1818

1919
AudioFile<T> audioFileWriter;
2020

@@ -47,12 +47,12 @@ void writeTestAudioFile (int numChannels, int sampleRate, int bitDepth, AudioFil
4747
T sample;
4848

4949
if constexpr (std::numeric_limits<T>::is_integer && std::is_unsigned_v<T>)
50-
sample = (T)(((sinf (2. * M_PI * ((double) i / sampleRateAsFloat) * 440.) + 1.) / 2.) * maxValue);
50+
sample = static_cast<T> (((sin (2. * M_PI * (static_cast<double> (i) / static_cast<double> (sampleRateAsFloat)) * 440.) + 1.) / 2.) * maxValue);
5151
else
52-
sample = (T)(sinf (2. * M_PI * ((double) i / sampleRateAsFloat) * 440.) * maxValue);
52+
sample = static_cast<T> (sin (2. * M_PI * (static_cast<double> (i) / static_cast<double> (sampleRateAsFloat)) * 440.) * static_cast<double> (maxValue));
5353

5454
for (int k = 0; k < audioFileWriter.getNumChannels(); k++)
55-
audioFileWriter.samples[k][i] = sample * 0.5f;
55+
audioFileWriter.samples[k][i] = sample * static_cast<T> (0.5);
5656
}
5757

5858
audioFileWriter.setSampleRate (sampleRate);

tests/SampleConversionTests.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ TEST_SUITE ("SampleConversionTests::8-bit Conversions - sampleToSignedByte()")
103103
REQUIRE_EQ (AudioSampleConverter<float>::sampleToSignedByte (-1.f), -127);
104104
REQUIRE_EQ (AudioSampleConverter<float>::sampleToSignedByte (std::numeric_limits<float>::lowest()), -127);
105105

106-
REQUIRE_EQ (AudioSampleConverter<double>::sampleToSignedByte (std::numeric_limits<float>::max()), 127);
106+
REQUIRE_EQ (AudioSampleConverter<double>::sampleToSignedByte (std::numeric_limits<double>::max()), 127);
107107
REQUIRE_EQ (AudioSampleConverter<double>::sampleToSignedByte (1.), 127);
108108
REQUIRE_EQ (AudioSampleConverter<double>::sampleToSignedByte (0.5), 63);
109109
REQUIRE_EQ (AudioSampleConverter<double>::sampleToSignedByte (0.), 0);
@@ -717,10 +717,10 @@ TEST_SUITE ("SampleConversionTests::32-bit Conversions")
717717
REQUIRE_EQ (AudioSampleConverter<float>::sampleToThirtyTwoBitInt (std::numeric_limits<float>::lowest()), -2147483647);
718718

719719
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (std::numeric_limits<double>::max()), 2147483647);
720-
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (1.f), 2147483647);
721-
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (0.5f), 1073741823);
722-
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (0.f), 0);
723-
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (-0.5f), -1073741823);
720+
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (1.), 2147483647);
721+
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (0.5), 1073741823);
722+
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (0.), 0);
723+
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (-0.5), -1073741823);
724724
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (-1.), -2147483647);
725725
REQUIRE_EQ (AudioSampleConverter<double>::sampleToThirtyTwoBitInt (std::numeric_limits<double>::lowest()), -2147483647);
726726
}

0 commit comments

Comments
 (0)