diff options
Diffstat (limited to 'src/audio_core')
| -rw-r--r-- | src/audio_core/CMakeLists.txt | 3 | ||||
| -rw-r--r-- | src/audio_core/audio_renderer.cpp | 8 | ||||
| -rw-r--r-- | src/audio_core/audio_renderer.h | 8 | ||||
| -rw-r--r-- | src/audio_core/command_generator.cpp | 4 | ||||
| -rw-r--r-- | src/audio_core/common.h | 4 | ||||
| -rw-r--r-- | src/audio_core/effect_context.cpp | 4 | ||||
| -rw-r--r-- | src/audio_core/info_updater.cpp | 5 | ||||
| -rw-r--r-- | src/audio_core/info_updater.h | 4 | ||||
| -rw-r--r-- | src/audio_core/splitter_context.cpp | 2 | ||||
| -rw-r--r-- | src/audio_core/voice_context.cpp | 4 |
10 files changed, 21 insertions, 25 deletions
diff --git a/src/audio_core/CMakeLists.txt b/src/audio_core/CMakeLists.txt index e553b8203..89575a53e 100644 --- a/src/audio_core/CMakeLists.txt +++ b/src/audio_core/CMakeLists.txt @@ -49,9 +49,6 @@ if (NOT MSVC) target_compile_options(audio_core PRIVATE -Werror=conversion -Werror=ignored-qualifiers - -Werror=shadow - -Werror=unused-parameter - -Werror=unused-variable $<$<CXX_COMPILER_ID:GNU>:-Werror=unused-but-set-parameter> $<$<CXX_COMPILER_ID:GNU>:-Werror=unused-but-set-variable> diff --git a/src/audio_core/audio_renderer.cpp b/src/audio_core/audio_renderer.cpp index e40ab16d2..2ee0a96ed 100644 --- a/src/audio_core/audio_renderer.cpp +++ b/src/audio_core/audio_renderer.cpp @@ -98,13 +98,13 @@ AudioRenderer::AudioRenderer(Core::Timing::CoreTiming& core_timing_, Core::Memor AudioRenderer::~AudioRenderer() = default; -ResultCode AudioRenderer::Start() { +Result AudioRenderer::Start() { audio_out->StartStream(stream); ReleaseAndQueueBuffers(); return ResultSuccess; } -ResultCode AudioRenderer::Stop() { +Result AudioRenderer::Stop() { audio_out->StopStream(stream); return ResultSuccess; } @@ -125,8 +125,8 @@ Stream::State AudioRenderer::GetStreamState() const { return stream->GetState(); } -ResultCode AudioRenderer::UpdateAudioRenderer(const std::vector<u8>& input_params, - std::vector<u8>& output_params) { +Result AudioRenderer::UpdateAudioRenderer(const std::vector<u8>& input_params, + std::vector<u8>& output_params) { std::scoped_lock lock{mutex}; InfoUpdater info_updater{input_params, output_params, behavior_info}; diff --git a/src/audio_core/audio_renderer.h b/src/audio_core/audio_renderer.h index 1f9f55ae2..a67ffd592 100644 --- a/src/audio_core/audio_renderer.h +++ b/src/audio_core/audio_renderer.h @@ -43,10 +43,10 @@ public: Stream::ReleaseCallback&& release_callback, std::size_t instance_number); ~AudioRenderer(); - [[nodiscard]] ResultCode UpdateAudioRenderer(const std::vector<u8>& input_params, - std::vector<u8>& output_params); - [[nodiscard]] ResultCode Start(); - [[nodiscard]] ResultCode Stop(); + [[nodiscard]] Result UpdateAudioRenderer(const std::vector<u8>& input_params, + std::vector<u8>& output_params); + [[nodiscard]] Result Start(); + [[nodiscard]] Result Stop(); void QueueMixedBuffer(Buffer::Tag tag); void ReleaseAndQueueBuffers(); [[nodiscard]] u32 GetSampleRate() const; diff --git a/src/audio_core/command_generator.cpp b/src/audio_core/command_generator.cpp index ff20ed00f..f97520820 100644 --- a/src/audio_core/command_generator.cpp +++ b/src/audio_core/command_generator.cpp @@ -429,7 +429,7 @@ void CommandGenerator::GenerateDataSourceCommand(ServerVoiceInfo& voice_info, Vo in_params.node_id); break; default: - UNREACHABLE_MSG("Unimplemented sample format={}", in_params.sample_format); + ASSERT_MSG(false, "Unimplemented sample format={}", in_params.sample_format); } } } @@ -1312,7 +1312,7 @@ void CommandGenerator::DecodeFromWaveBuffers(ServerVoiceInfo& voice_info, std::s samples_to_read - samples_read, channel, temp_mix_offset); break; default: - UNREACHABLE_MSG("Unimplemented sample format={}", in_params.sample_format); + ASSERT_MSG(false, "Unimplemented sample format={}", in_params.sample_format); } temp_mix_offset += samples_decoded; diff --git a/src/audio_core/common.h b/src/audio_core/common.h index 46ef83113..056a0ac70 100644 --- a/src/audio_core/common.h +++ b/src/audio_core/common.h @@ -10,8 +10,8 @@ namespace AudioCommon { namespace Audren { -constexpr ResultCode ERR_INVALID_PARAMETERS{ErrorModule::Audio, 41}; -constexpr ResultCode ERR_SPLITTER_SORT_FAILED{ErrorModule::Audio, 43}; +constexpr Result ERR_INVALID_PARAMETERS{ErrorModule::Audio, 41}; +constexpr Result ERR_SPLITTER_SORT_FAILED{ErrorModule::Audio, 43}; } // namespace Audren constexpr u8 BASE_REVISION = '0'; diff --git a/src/audio_core/effect_context.cpp b/src/audio_core/effect_context.cpp index 51059580e..79bcd1192 100644 --- a/src/audio_core/effect_context.cpp +++ b/src/audio_core/effect_context.cpp @@ -50,7 +50,7 @@ EffectBase* EffectContext::RetargetEffect(std::size_t i, EffectType effect) { effects[i] = std::make_unique<EffectBiquadFilter>(); break; default: - UNREACHABLE_MSG("Unimplemented effect {}", effect); + ASSERT_MSG(false, "Unimplemented effect {}", effect); effects[i] = std::make_unique<EffectStubbed>(); } return GetInfo(i); @@ -104,7 +104,7 @@ void EffectI3dl2Reverb::Update(EffectInfo::InParams& in_params) { auto& params = GetParams(); const auto* reverb_params = reinterpret_cast<I3dl2ReverbParams*>(in_params.raw.data()); if (!ValidChannelCountForEffect(reverb_params->max_channels)) { - UNREACHABLE_MSG("Invalid reverb max channel count {}", reverb_params->max_channels); + ASSERT_MSG(false, "Invalid reverb max channel count {}", reverb_params->max_channels); return; } diff --git a/src/audio_core/info_updater.cpp b/src/audio_core/info_updater.cpp index 313a2eb6d..0065e6e53 100644 --- a/src/audio_core/info_updater.cpp +++ b/src/audio_core/info_updater.cpp @@ -285,9 +285,8 @@ bool InfoUpdater::UpdateSplitterInfo(SplitterContext& splitter_context) { return true; } -ResultCode InfoUpdater::UpdateMixes(MixContext& mix_context, std::size_t mix_buffer_count, - SplitterContext& splitter_context, - EffectContext& effect_context) { +Result InfoUpdater::UpdateMixes(MixContext& mix_context, std::size_t mix_buffer_count, + SplitterContext& splitter_context, EffectContext& effect_context) { std::vector<MixInfo::InParams> mix_in_params; if (!behavior_info.IsMixInParameterDirtyOnlyUpdateSupported()) { diff --git a/src/audio_core/info_updater.h b/src/audio_core/info_updater.h index 6aab5beaf..17e66b036 100644 --- a/src/audio_core/info_updater.h +++ b/src/audio_core/info_updater.h @@ -32,8 +32,8 @@ public: VAddr audio_codec_dsp_addr); bool UpdateEffects(EffectContext& effect_context, bool is_active); bool UpdateSplitterInfo(SplitterContext& splitter_context); - ResultCode UpdateMixes(MixContext& mix_context, std::size_t mix_buffer_count, - SplitterContext& splitter_context, EffectContext& effect_context); + Result UpdateMixes(MixContext& mix_context, std::size_t mix_buffer_count, + SplitterContext& splitter_context, EffectContext& effect_context); bool UpdateSinks(SinkContext& sink_context); bool UpdatePerformanceBuffer(); bool UpdateErrorInfo(BehaviorInfo& in_behavior_info); diff --git a/src/audio_core/splitter_context.cpp b/src/audio_core/splitter_context.cpp index 1751d0212..10646dc05 100644 --- a/src/audio_core/splitter_context.cpp +++ b/src/audio_core/splitter_context.cpp @@ -483,7 +483,7 @@ bool NodeStates::DepthFirstSearch(EdgeMatrix& edge_matrix) { // Add more work index_stack.push(j); } else if (node_state == NodeStates::State::InFound) { - UNREACHABLE_MSG("Node start marked as found"); + ASSERT_MSG(false, "Node start marked as found"); ResetState(); return false; } diff --git a/src/audio_core/voice_context.cpp b/src/audio_core/voice_context.cpp index c8e4a6caf..f58a5c754 100644 --- a/src/audio_core/voice_context.cpp +++ b/src/audio_core/voice_context.cpp @@ -114,7 +114,7 @@ void ServerVoiceInfo::UpdateParameters(const VoiceInfo::InParams& voice_in, in_params.current_playstate = ServerPlayState::Play; break; default: - UNREACHABLE_MSG("Unknown playstate {}", voice_in.play_state); + ASSERT_MSG(false, "Unknown playstate {}", voice_in.play_state); break; } @@ -410,7 +410,7 @@ bool ServerVoiceInfo::UpdateParametersForCommandGeneration( return in_params.should_depop; } default: - UNREACHABLE_MSG("Invalid playstate {}", in_params.current_playstate); + ASSERT_MSG(false, "Invalid playstate {}", in_params.current_playstate); } return false; |
