Don't say 'certificate downloaded' if it failed during the read part.

[dcpomatic.git] / src / lib / audio_merger.cc

diff --git a/src/lib/audio_merger.cc b/src/lib/audio_merger.cc
index 4bed4b3a5850539f8617f3ecb4a348889eacb2a8..c1d5a54dd88e4c713785cdf786aff7f60a227fc3 100644 (file)
--- a/src/lib/audio_merger.cc
+++ b/src/lib/audio_merger.cc
@@ -34,10 +34,14 @@ using std::make_pair;
 using boost::shared_ptr;
 using boost::optional;
 
+// #define INSTRUMENT 1
+
 AudioMerger::AudioMerger (int frame_rate)
        : _frame_rate (frame_rate)
 {
-
+#ifdef INSTRUMENT
+       cout << "I/AM frame_rate " << frame_rate << "\n";
+#endif
 }
 
 Frame
@@ -54,32 +58,44 @@ AudioMerger::frames (DCPTime t) const
 list<pair<shared_ptr<AudioBuffers>, DCPTime> >
 AudioMerger::pull (DCPTime time)
 {
+#ifdef INSTRUMENT
+       std::cout << "I/AM pull " << time.get() << "\n";
+#endif
        list<pair<shared_ptr<AudioBuffers>, DCPTime> > out;
 
-       DCPTimePeriod period (_last_pull, time);
-       _buffers.sort (AudioMerger::BufferComparator());
-
        list<Buffer> new_buffers;
 
+       _buffers.sort (AudioMerger::BufferComparator());
        BOOST_FOREACH (Buffer i, _buffers) {
                if (i.period().to <= time) {
                        /* Completely within the pull period */
+                       DCPOMATIC_ASSERT (i.audio->frames() > 0);
                        out.push_back (make_pair (i.audio, i.time));
                } else if (i.time < time) {
                        /* Overlaps the end of the pull period */
                        shared_ptr<AudioBuffers> audio (new AudioBuffers (i.audio->channels(), frames(DCPTime(time - i.time))));
-                       audio->copy_from (i.audio.get(), audio->frames(), 0, 0);
-                       out.push_back (make_pair (audio, i.time));
-                       i.audio->trim_start (audio->frames ());
-                       i.time += DCPTime::from_frames(audio->frames(), _frame_rate);
-                       new_buffers.push_back (i);
+                       /* Though time > i.time, audio->frames() could be 0 if the difference in time is less than one frame */
+                       if (audio->frames() > 0) {
+                               audio->copy_from (i.audio.get(), audio->frames(), 0, 0);
+                               out.push_back (make_pair (audio, i.time));
+                               i.audio->trim_start (audio->frames ());
+                               i.time += DCPTime::from_frames(audio->frames(), _frame_rate);
+                               DCPOMATIC_ASSERT (i.audio->frames() > 0);
+                               new_buffers.push_back (i);
+                       }
                } else {
                        /* Not involved */
+                       DCPOMATIC_ASSERT (i.audio->frames() > 0);
                        new_buffers.push_back (i);
                }
        }
 
        _buffers = new_buffers;
+
+       for (list<pair<shared_ptr<AudioBuffers>, DCPTime> >::const_iterator i = out.begin(); i != out.end(); ++i) {
+               DCPOMATIC_ASSERT (i->first->frames() > 0);
+       }
+
        return out;
 }
 
@@ -87,7 +103,10 @@ AudioMerger::pull (DCPTime time)
 void
 AudioMerger::push (boost::shared_ptr<const AudioBuffers> audio, DCPTime time)
 {
-       DCPOMATIC_ASSERT (time >= _last_pull);
+#ifdef INSTRUMENT
+       std::cout << "I/AM push " << time.get() << " " << audio->frames() << "\n";
+#endif
+       DCPOMATIC_ASSERT (audio->frames() > 0);
 
        DCPTimePeriod period (time, time + DCPTime::from_frames (audio->frames(), _frame_rate));
 
@@ -128,8 +147,10 @@ AudioMerger::push (boost::shared_ptr<const AudioBuffers> audio, DCPTime time)
                part->copy_from (audio.get(), part->frames(), frames(DCPTime(i.from - time)), 0);
 
                if (before == _buffers.end() && after == _buffers.end()) {
-                       /* New buffer */
-                       _buffers.push_back (Buffer (part, time, _frame_rate));
+                       if (part->frames() > 0) {
+                               /* New buffer */
+                               _buffers.push_back (Buffer (part, time, _frame_rate));
+                       }
                } else if (before != _buffers.end() && after == _buffers.end()) {
                        /* We have an existing buffer before this one; append new data to it */
                        before->audio->append (part);
@@ -146,3 +167,12 @@ AudioMerger::push (boost::shared_ptr<const AudioBuffers> audio, DCPTime time)
                }
        }
 }
+
+void
+AudioMerger::clear ()
+{
+#ifdef INSTRUMENT
+       cout << "I/AM clear\n";
+#endif
+       _buffers.clear ();
+}