Conversion of frame index to FFmpeg time must always use the original or detected...

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

diff --git a/src/lib/sndfile_decoder.cc b/src/lib/sndfile_decoder.cc
index dd9e654c77de7984a4672a7a577fb8d91f340f44..f66a7c7dc9dd363f649f2c71632a9a5057d78949 100644 (file)
--- a/src/lib/sndfile_decoder.cc
+++ b/src/lib/sndfile_decoder.cc
@@ -18,11 +18,16 @@
 */
 
 #include <iostream>
+#ifdef DCPOMATIC_WINDOWS
+#include <windows.h>
+#define ENABLE_SNDFILE_WINDOWS_PROTOTYPES 1
+#endif
 #include <sndfile.h>
 #include "sndfile_content.h"
 #include "sndfile_decoder.h"
 #include "film.h"
 #include "exceptions.h"
+#include "audio_buffers.h"
 
 #include "i18n.h"
 
@@ -34,11 +39,19 @@ using boost::shared_ptr;
 
 SndfileDecoder::SndfileDecoder (shared_ptr<const Film> f, shared_ptr<const SndfileContent> c)
        : Decoder (f)
-       , AudioDecoder (f)
+       , AudioDecoder (f, c)
        , _sndfile_content (c)
        , _deinterleave_buffer (0)
 {
-       _sndfile = sf_open (_sndfile_content->file().string().c_str(), SFM_READ, &_info);
+       _info.format = 0;
+
+       /* Here be monsters.  See fopen_boost for similar shenanigans */
+#ifdef DCPOMATIC_WINDOWS
+       _sndfile = sf_wchar_open (_sndfile_content->path(0).c_str(), SFM_READ, &_info);
+#else  
+       _sndfile = sf_open (_sndfile_content->path(0).string().c_str(), SFM_READ, &_info);
+#endif
+       
        if (!_sndfile) {
                throw DecodeError (_("could not open audio file for reading"));
        }
@@ -53,22 +66,22 @@ SndfileDecoder::~SndfileDecoder ()
        delete[] _deinterleave_buffer;
 }
 
-bool
+void
 SndfileDecoder::pass ()
 {
        /* Do things in half second blocks as I think there may be limits
           to what FFmpeg (and in particular the resampler) can cope with.
        */
-       sf_count_t const block = _sndfile_content->audio_frame_rate() / 2;
+       sf_count_t const block = _sndfile_content->content_audio_frame_rate() / 2;
        sf_count_t const this_time = min (block, _remaining);
 
        int const channels = _sndfile_content->audio_channels ();
        
-       shared_ptr<AudioBuffers> audio (new AudioBuffers (channels, this_time));
+       shared_ptr<AudioBuffers> data (new AudioBuffers (channels, this_time));
 
        if (_sndfile_content->audio_channels() == 1) {
                /* No de-interleaving required */
-               sf_read_float (_sndfile, audio->data(0), this_time);
+               sf_read_float (_sndfile, data->data(0), this_time);
        } else {
                /* Deinterleave */
                if (!_deinterleave_buffer) {
@@ -77,7 +90,7 @@ SndfileDecoder::pass ()
                sf_readf_float (_sndfile, _deinterleave_buffer, this_time);
                vector<float*> out_ptr (channels);
                for (int i = 0; i < channels; ++i) {
-                       out_ptr[i] = audio->data(i);
+                       out_ptr[i] = data->data(i);
                }
                float* in_ptr = _deinterleave_buffer;
                for (int i = 0; i < this_time; ++i) {
@@ -87,12 +100,10 @@ SndfileDecoder::pass ()
                }
        }
                
-       audio->set_frames (this_time);
-       Audio (audio, double(_done) / audio_frame_rate());
+       data->set_frames (this_time);
+       audio (data, _done);
        _done += this_time;
        _remaining -= this_time;
-
-       return (_remaining == 0);
 }
 
 int
@@ -101,7 +112,7 @@ SndfileDecoder::audio_channels () const
        return _info.channels;
 }
 
-ContentAudioFrame
+AudioContent::Frame
 SndfileDecoder::audio_length () const
 {
        return _info.frames;
@@ -112,3 +123,9 @@ SndfileDecoder::audio_frame_rate () const
 {
        return _info.samplerate;
 }
+
+bool
+SndfileDecoder::done () const
+{
+       return _audio_position >= _sndfile_content->audio_length ();
+}