Merge branch 'master' into plot-audio
[dcpomatic.git] / src / lib / writer.cc
index 5bd32147a65e81ed947168ee9d888bfdf19213c3..1dad4357d9381f02f5a9b9e3382c22c0a6a71153 100644 (file)
@@ -33,18 +33,27 @@ using std::make_pair;
 using std::pair;
 using std::string;
 using std::ifstream;
+using std::list;
 using std::cout;
 using boost::shared_ptr;
 
-unsigned int const Writer::_maximum_frames_in_memory = 8;
+int const Writer::_maximum_frames_in_memory = 8;
 
 Writer::Writer (shared_ptr<Film> f)
        : _film (f)
        , _first_nonexistant_frame (0)
        , _thread (0)
        , _finish (false)
+       , _queued_full_in_memory (0)
        , _last_written_frame (-1)
+       , _full_written (0)
+       , _fake_written (0)
+       , _repeat_written (0)
+       , _pushed_to_disk (0)
 {
+       /* Remove any old DCP */
+       boost::filesystem::remove_all (_film->dir (_film->dcp_name ()));
+       
        check_existing_picture_mxf ();
        
        /* Create our picture asset in a subdirectory, named according to those
@@ -63,20 +72,22 @@ Writer::Writer (shared_ptr<Film> f)
 
        _picture_asset_writer = _picture_asset->start_write (_first_nonexistant_frame > 0);
 
-       if (_film->audio_channels() > 0) {
+       AudioMapping m (_film->audio_channels ());
+       
+       if (m.dcp_channels() > 0) {
                _sound_asset.reset (
                        new libdcp::SoundAsset (
                                _film->dir (_film->dcp_name()),
                                "audio.mxf",
                                DCPFrameRate (_film->frames_per_second()).frames_per_second,
-                               _film->audio_channels(),
+                               m.dcp_channels (),
                                dcp_audio_sample_rate (_film->audio_stream()->sample_rate())
                                )
                        );
 
                _sound_asset_writer = _sound_asset->start_write ();
        }
-       
+
        _thread = new boost::thread (boost::bind (&Writer::thread, this));
 }
 
@@ -90,6 +101,7 @@ Writer::write (shared_ptr<const EncodedData> encoded, int frame)
        qi.encoded = encoded;
        qi.frame = frame;
        _queue.push_back (qi);
+       ++_queued_full_in_memory;
 
        _condition.notify_all ();
 }
@@ -120,27 +132,29 @@ Writer::write (shared_ptr<const AudioBuffers> audio)
 
 void
 Writer::thread ()
+try
 {
        while (1)
        {
                boost::mutex::scoped_lock lock (_mutex);
 
                while (1) {
+                       
+                       _queue.sort ();
+                       
                        if (_finish ||
-                           _queue.size() > _maximum_frames_in_memory ||
+                           _queued_full_in_memory > _maximum_frames_in_memory ||
                            (!_queue.empty() && _queue.front().frame == (_last_written_frame + 1))) {
-                                   
-                                   break;
-                           }
-
-                           TIMING ("writer sleeps with a queue of %1; %2 pending", _queue.size(), _pending.size());
-                           _condition.wait (lock);
-                           TIMING ("writer wakes with a queue of %1", _queue.size());
-
-                           _queue.sort ();
+                               
+                               break;
+                       }
+                       
+                       TIMING ("writer sleeps with a queue of %1", _queue.size());
+                       _condition.wait (lock);
+                       TIMING ("writer wakes with a queue of %1", _queue.size());
                }
 
-               if (_finish && _queue.empty() && _pending.empty()) {
+               if (_finish && _queue.empty()) {
                        return;
                }
 
@@ -148,27 +162,36 @@ Writer::thread ()
                while (!_queue.empty() && _queue.front().frame == (_last_written_frame + 1)) {
                        QueueItem qi = _queue.front ();
                        _queue.pop_front ();
+                       if (qi.type == QueueItem::FULL && qi.encoded) {
+                               --_queued_full_in_memory;
+                       }
 
                        lock.unlock ();
                        switch (qi.type) {
                        case QueueItem::FULL:
                        {
                                _film->log()->log (String::compose ("Writer FULL-writes %1 to MXF", qi.frame));
+                               if (!qi.encoded) {
+                                       qi.encoded.reset (new EncodedData (_film->j2c_path (qi.frame, false)));
+                               }
                                libdcp::FrameInfo const fin = _picture_asset_writer->write (qi.encoded->data(), qi.encoded->size());
                                qi.encoded->write_info (_film, qi.frame, fin);
                                _last_written = qi.encoded;
+                               ++_full_written;
                                break;
                        }
                        case QueueItem::FAKE:
                                _film->log()->log (String::compose ("Writer FAKE-writes %1 to MXF", qi.frame));
                                _picture_asset_writer->fake_write (qi.size);
                                _last_written.reset ();
+                               ++_fake_written;
                                break;
                        case QueueItem::REPEAT:
                        {
                                _film->log()->log (String::compose ("Writer REPEAT-writes %1 to MXF", qi.frame));
                                libdcp::FrameInfo const fin = _picture_asset_writer->write (_last_written->data(), _last_written->size());
                                _last_written->write_info (_film, qi.frame, fin);
+                               ++_repeat_written;
                                break;
                        }
                        }
@@ -177,44 +200,34 @@ Writer::thread ()
                        ++_last_written_frame;
                }
 
-               while (_queue.size() > _maximum_frames_in_memory) {
+               while (_queued_full_in_memory > _maximum_frames_in_memory) {
                        /* Too many frames in memory which can't yet be written to the stream.
-                          Put some in our pending list (and write FULL queue items' data to disk)
+                          Write some FULL frames to disk.
                        */
 
-                       QueueItem qi = _queue.back ();
-                       _queue.pop_back ();
-
-                       if (qi.type == QueueItem::FULL) {
-                               lock.unlock ();
-                               _film->log()->log (String::compose ("Writer full (awaiting %1); pushes %2 to disk", _last_written_frame + 1, qi.frame));
-                               qi.encoded->write (_film, qi.frame);
-                               lock.lock ();
-                               qi.encoded.reset ();
+                       /* Find one */
+                       list<QueueItem>::reverse_iterator i = _queue.rbegin ();
+                       while (i != _queue.rend() && (i->type != QueueItem::FULL || !i->encoded)) {
+                               ++i;
                        }
 
-                       _pending.push_back (qi);
-               }
-
-               while (_queue.size() < _maximum_frames_in_memory && !_pending.empty()) {
-                       /* We have some space in memory.  Fetch some frames back off disk. */
-
-                       _pending.sort ();
-                       QueueItem qi = _pending.front ();
+                       assert (i != _queue.rend());
+                       QueueItem qi = *i;
 
-                       if (qi.type == QueueItem::FULL) {
-                               lock.unlock ();
-                               _film->log()->log (String::compose ("Writer pulls %1 back from disk", qi.frame));
-                               shared_ptr<const EncodedData> encoded;
-                               qi.encoded.reset (new EncodedData (_film->j2c_path (qi.frame, false)));
-                               lock.lock ();
-                       }
-
-                       _queue.push_back (qi);
-                       _pending.remove (qi);
+                       ++_pushed_to_disk;
+                       
+                       lock.unlock ();
+                       _film->log()->log (String::compose ("Writer full (awaiting %1); pushes %2 to disk", _last_written_frame + 1, qi.frame));
+                       qi.encoded->write (_film, qi.frame);
+                       lock.lock ();
+                       qi.encoded.reset ();
+                       --_queued_full_in_memory;
                }
        }
-
+}
+catch (...)
+{
+       store_current ();
 }
 
 void
@@ -230,6 +243,10 @@ Writer::finish ()
        lock.unlock ();
 
        _thread->join ();
+       if (thrown ()) {
+               rethrow ();
+       }
+       
        delete _thread;
        _thread = 0;
 
@@ -286,6 +303,8 @@ Writer::finish ()
                               ));
 
        dcp.write_xml ();
+
+       _film->log()->log (String::compose ("Wrote %1 FULL, %2 FAKE, %3 REPEAT; %4 pushed to disk", _full_written, _fake_written, _repeat_written, _pushed_to_disk));
 }
 
 /** Tell the writer that frame `f' should be a repeat of the frame before it */
@@ -340,14 +359,18 @@ Writer::check_existing_picture_mxf ()
        fclose (mxf);
 }
 
-/** @return true if the fake write succeeded, otherwise false */
+/** @param frame Frame index.
+ *  @return true if we can fake-write this frame.
+ */
 bool
 Writer::can_fake_write (int frame) const
 {
+       /* We have to do a proper write of the first frame so that we can set up the JPEG2000
+          parameters in the MXF writer.
+       */
        return (frame != 0 && frame < _first_nonexistant_frame);
 }
 
-
 bool
 operator< (QueueItem const & a, QueueItem const & b)
 {