From: Helge Norberg <helge.norberg@svt.se>
Date: Thu, 13 Oct 2016 16:00:03 +0000 (+0200)
Subject: [ffmpeg_producer] Moved negation from method to operator ! for clarity.
X-Git-Tag: 2.1.0_Beta1~26
X-Git-Url: https://git.sesse.net/?p=casparcg;a=commitdiff_plain;h=b59ec2b89fbc8a2fbccaaa42d623db97a241a54b

[ffmpeg_producer] Moved negation from method to operator ! for clarity.
---

diff --git a/modules/ffmpeg/producer/ffmpeg_producer.cpp b/modules/ffmpeg/producer/ffmpeg_producer.cpp
index a2d4afef0..48facf644 100644
--- a/modules/ffmpeg/producer/ffmpeg_producer.cpp
+++ b/modules/ffmpeg/producer/ffmpeg_producer.cpp
@@ -521,20 +521,20 @@ public:
 				!video_decoder_->is_progressive()) : L"";
 	}
 
-	bool not_all_audio_decoders_ready() const
+	bool all_audio_decoders_ready() const
 	{
 		for (auto& audio_decoder : audio_decoders_)
 			if (!audio_decoder->ready())
-				return true;
+				return false;
 
-		return false;
+		return true;
 	}
 
 	void try_decode_frame()
 	{
 		std::shared_ptr<AVPacket> pkt;
 
-		for (int n = 0; n < 32 && ((video_decoder_ && !video_decoder_->ready()) || not_all_audio_decoders_ready()) && input_.try_pop(pkt); ++n)
+		for (int n = 0; n < 32 && ((video_decoder_ && !video_decoder_->ready()) || !all_audio_decoders_ready()) && input_.try_pop(pkt); ++n)
 		{
 			if (video_decoder_)
 				video_decoder_->push(pkt);