Commit 7ff5cf83 authored by Max Kellermann's avatar Max Kellermann

filter/convert: add method IsActive()

parent eb771eaf
...@@ -59,6 +59,11 @@ public: ...@@ -59,6 +59,11 @@ public:
ConstBuffer<void> Flush() override { ConstBuffer<void> Flush() override {
return state.Flush(); return state.Flush();
} }
private:
bool IsActive() const noexcept {
return out_audio_format != in_audio_format;
}
}; };
class PreparedConvertFilter final : public PreparedFilter { class PreparedConvertFilter final : public PreparedFilter {
...@@ -76,7 +81,7 @@ ConvertFilter::Set(const AudioFormat &_out_audio_format) ...@@ -76,7 +81,7 @@ ConvertFilter::Set(const AudioFormat &_out_audio_format)
/* no change */ /* no change */
return; return;
if (out_audio_format != in_audio_format) { if (IsActive()) {
out_audio_format = in_audio_format; out_audio_format = in_audio_format;
state.Close(); state.Close();
} }
...@@ -107,7 +112,7 @@ ConvertFilter::~ConvertFilter() ...@@ -107,7 +112,7 @@ ConvertFilter::~ConvertFilter()
{ {
assert(in_audio_format.IsValid()); assert(in_audio_format.IsValid());
if (out_audio_format != in_audio_format) if (IsActive())
state.Close(); state.Close();
} }
...@@ -116,11 +121,10 @@ ConvertFilter::FilterPCM(ConstBuffer<void> src) ...@@ -116,11 +121,10 @@ ConvertFilter::FilterPCM(ConstBuffer<void> src)
{ {
assert(in_audio_format.IsValid()); assert(in_audio_format.IsValid());
if (out_audio_format == in_audio_format) return IsActive()
? state.Convert(src)
/* optimized special case: no-op */ /* optimized special case: no-op */
return src; : src;
return state.Convert(src);
} }
std::unique_ptr<PreparedFilter> std::unique_ptr<PreparedFilter>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment