Skip to content

Commit 7ccbc2b

Browse files
committed
Fix warnings in tests
1 parent 952f8b2 commit 7ccbc2b

File tree

11 files changed

+21
-12
lines changed

11 files changed

+21
-12
lines changed

modules/yup_audio_devices/yup_audio_devices.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,6 @@
6262

6363
#include "yup_audio_devices.h"
6464

65-
#include "midi_io/ump/yup_UMPBytestreamInputHandler.h"
66-
#include "midi_io/ump/yup_UMPU32InputHandler.h"
67-
6865
#include "midi_io/yup_MidiDevices.cpp"
6966
#include "audio_io/yup_SampleRateHelpers.cpp"
7067

modules/yup_python/scripting/yup_ScriptUtilities.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ std::optional<T> python_cast (const pybind11::object& value)
5555
5656
@param scriptEngine The script engine to redirect the streams to.
5757
*/
58-
struct YUP_API ScriptStreamRedirection
58+
struct ScriptStreamRedirection
5959
{
6060
ScriptStreamRedirection() noexcept;
6161
~ScriptStreamRedirection() noexcept;

tests/yup_audio_basics/yup_AudioPlayHead.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ class TestAudioPlayHead : public AudioPlayHead
4949
private:
5050
std::optional<PositionInfo> testPosition;
5151
};
52-
} // namespace
5352

5453
class AudioPlayHeadTests : public ::testing::Test
5554
{
@@ -62,6 +61,8 @@ class AudioPlayHeadTests : public ::testing::Test
6261
std::unique_ptr<TestAudioPlayHead> playHead;
6362
};
6463

64+
} // namespace
65+
6566
TEST_F (AudioPlayHeadTests, DefaultTransportControlMethodsExist)
6667
{
6768
// Test that default implementations exist and don't crash

tests/yup_audio_basics/yup_BufferingAudioSource.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,6 @@ class MockPositionableAudioSource : public PositionableAudioSource
104104
int64 currentPosition;
105105
bool looping;
106106
};
107-
} // namespace
108107

109108
//==============================================================================
110109
class BufferingAudioSourceTests : public ::testing::Test
@@ -131,6 +130,8 @@ class BufferingAudioSourceTests : public ::testing::Test
131130
std::unique_ptr<BufferingAudioSource> buffering;
132131
};
133132

133+
} // namespace
134+
134135
//==============================================================================
135136
TEST_F (BufferingAudioSourceTests, Constructor)
136137
{

tests/yup_audio_basics/yup_ChannelRemappingAudioSource.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ class MockAudioSource : public AudioSource
6767
int lastSamplesPerBlock = 0;
6868
double lastSampleRate = 0.0;
6969
};
70-
} // namespace
7170

7271
//==============================================================================
7372
class ChannelRemappingAudioSourceTests : public ::testing::Test
@@ -88,6 +87,8 @@ class ChannelRemappingAudioSourceTests : public ::testing::Test
8887
std::unique_ptr<ChannelRemappingAudioSource> remapper;
8988
};
9089

90+
} // namespace
91+
9192
//==============================================================================
9293
TEST_F (ChannelRemappingAudioSourceTests, Constructor)
9394
{

tests/yup_audio_basics/yup_MidiKeyboardState.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ class TestListener : public MidiKeyboardState::Listener
5757
std::vector<NoteEvent> noteOnCalls;
5858
std::vector<NoteEvent> noteOffCalls;
5959
};
60-
} // namespace
6160

6261
//==============================================================================
6362
class MidiKeyboardStateTests : public ::testing::Test
@@ -79,6 +78,8 @@ class MidiKeyboardStateTests : public ::testing::Test
7978
std::unique_ptr<TestListener> listener;
8079
};
8180

81+
} // namespace
82+
8283
//==============================================================================
8384
// Constructor and Reset Tests
8485
//==============================================================================

tests/yup_audio_basics/yup_ResamplingAudioSource.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ class MockResamplingAudioSource : public AudioSource
6767
int lastSamplesPerBlock = 0;
6868
double lastSampleRate = 0.0;
6969
};
70-
} // namespace
7170

7271
//==============================================================================
7372
class ResamplingAudioSourceTests : public ::testing::Test
@@ -88,6 +87,8 @@ class ResamplingAudioSourceTests : public ::testing::Test
8887
std::unique_ptr<ResamplingAudioSource> resampler;
8988
};
9089

90+
} // namespace
91+
9192
//==============================================================================
9293
TEST_F (ResamplingAudioSourceTests, Constructor)
9394
{

tests/yup_audio_basics/yup_ReverbAudioSource.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ class MockReverbAudioSource : public AudioSource
6767
double lastSampleRate = 0.0;
6868
float fillValue = 0.5f;
6969
};
70-
} // namespace
7170

7271
//==============================================================================
7372
class ReverbAudioSourceTests : public ::testing::Test
@@ -88,6 +87,8 @@ class ReverbAudioSourceTests : public ::testing::Test
8887
std::unique_ptr<ReverbAudioSource> reverbSource;
8988
};
9089

90+
} // namespace
91+
9192
//==============================================================================
9293
TEST_F (ReverbAudioSourceTests, ConstructorWithDeleteInput)
9394
{

tests/yup_audio_basics/yup_Synthesiser.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -204,8 +204,6 @@ class TestVoice : public SynthesiserVoice
204204
SynthesiserSound* currentSound = nullptr;
205205
};
206206

207-
} // namespace
208-
209207
class SynthesiserTest : public ::testing::Test
210208
{
211209
protected:
@@ -223,6 +221,8 @@ class SynthesiserTest : public ::testing::Test
223221
std::unique_ptr<TestSynthesiser> synth;
224222
};
225223

224+
} // namespace
225+
226226
TEST_F (SynthesiserTest, DefaultConstruction)
227227
{
228228
Synthesiser synthesiser;

tests/yup_core/yup_ListenerList.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ using namespace yup;
4646

4747
namespace
4848
{
49+
4950
class MockListener
5051
{
5152
public:

0 commit comments

Comments
 (0)