Another Windows build fix.
[dcpomatic.git] / src / lib / image_decoder.cc
index d0973f1227afdf51dbbc495f0d55651337b3a639..fd51c1ba3963888ec891fc44e19349407a29c507 100644 (file)
@@ -39,39 +39,47 @@ using dcp::Size;
 
 ImageDecoder::ImageDecoder (shared_ptr<const ImageContent> c, shared_ptr<Log> log)
        : _image_content (c)
-       , _video_position (0)
+       , _frame_video_position (0)
 {
        video.reset (new VideoDecoder (this, c, log));
 }
 
 bool
-ImageDecoder::pass (PassReason, bool)
+ImageDecoder::pass ()
 {
-       if (_video_position >= _image_content->video->length()) {
+       if (_frame_video_position >= _image_content->video->length()) {
                return true;
        }
 
        if (!_image_content->still() || !_image) {
                /* Either we need an image or we are using moving images, so load one */
-               boost::filesystem::path path = _image_content->path (_image_content->still() ? 0 : _video_position);
+               boost::filesystem::path path = _image_content->path (_image_content->still() ? 0 : _frame_video_position);
                if (valid_j2k_file (path)) {
+                       AVPixelFormat pf;
+                       if (_image_content->video->colour_conversion()) {
+                               /* We have a specified colour conversion: assume the image is RGB */
+                               pf = AV_PIX_FMT_RGB48LE;
+                       } else {
+                               /* No specified colour conversion: assume the image is XYZ */
+                               pf = AV_PIX_FMT_XYZ12LE;
+                       }
                        /* We can't extract image size from a JPEG2000 codestream without decoding it,
                           so pass in the image content's size here.
                        */
-                       _image.reset (new J2KImageProxy (path, _image_content->video->size ()));
+                       _image.reset (new J2KImageProxy (path, _image_content->video->size(), pf));
                } else {
                        _image.reset (new MagickImageProxy (path));
                }
        }
 
-       video->give (_image, _video_position);
-       ++_video_position;
+       video->emit (_image, _frame_video_position);
+       ++_frame_video_position;
        return false;
 }
 
 void
 ImageDecoder::seek (ContentTime time, bool accurate)
 {
-       video->seek (time, accurate);
-       _video_position = time.frames_round (_image_content->active_video_frame_rate ());
+       Decoder::seek (time, accurate);
+       _frame_video_position = time.frames_round (_image_content->active_video_frame_rate ());
 }