projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Fix incorrect alpha step used for RGBA64 sources.
[dcpomatic.git]
/
src
/
lib
/
image_decoder.cc
diff --git
a/src/lib/image_decoder.cc
b/src/lib/image_decoder.cc
index 6d8f878efeef1a8fe0dd1cd44bed12708c11849b..26a9ad624b31fb168633c7aa3f721f44edf403db 100644
(file)
--- a/
src/lib/image_decoder.cc
+++ b/
src/lib/image_decoder.cc
@@
-1,5
+1,5
@@
/*
/*
- Copyright (C) 2012-20
16
Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-20
21
Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
This file is part of DCP-o-matic.
@@
-18,42
+18,49
@@
*/
*/
+
+#include "exceptions.h"
+#include "ffmpeg_image_proxy.h"
+#include "film.h"
+#include "frame_interval_checker.h"
+#include "image.h"
#include "image_content.h"
#include "image_decoder.h"
#include "image_content.h"
#include "image_decoder.h"
-#include "video_decoder.h"
-#include "image.h"
-#include "magick_image_proxy.h"
#include "j2k_image_proxy.h"
#include "j2k_image_proxy.h"
-#include "film.h"
-#include "exceptions.h"
+#include "util.h"
#include "video_content.h"
#include "video_content.h"
-#include
<Magick++.h>
+#include
"video_decoder.h"
#include <boost/filesystem.hpp>
#include <iostream>
#include "i18n.h"
#include <boost/filesystem.hpp>
#include <iostream>
#include "i18n.h"
+
using std::cout;
using std::cout;
-using boost::shared_ptr;
+using std::make_shared;
+using std::shared_ptr;
using dcp::Size;
using dcp::Size;
+using namespace dcpomatic;
+
-ImageDecoder::ImageDecoder (shared_ptr<const
ImageContent> c, shared_ptr<Log> log
)
- :
_image_content (c
)
- , _
frame_video_position (0
)
+ImageDecoder::ImageDecoder (shared_ptr<const
Film> film, shared_ptr<const ImageContent> c
)
+ :
Decoder (film
)
+ , _
image_content (c
)
{
{
- video
.reset (new VideoDecoder (this, c, log)
);
+ video
= make_shared<VideoDecoder>(this, c
);
}
}
-void
+
+bool
ImageDecoder::pass ()
{
if (_frame_video_position >= _image_content->video->length()) {
ImageDecoder::pass ()
{
if (_frame_video_position >= _image_content->video->length()) {
- return;
+ return
true
;
}
if (!_image_content->still() || !_image) {
/* Either we need an image or we are using moving images, so load one */
}
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 : _frame_video_position);
+
auto
path = _image_content->path (_image_content->still() ? 0 : _frame_video_position);
if (valid_j2k_file (path)) {
AVPixelFormat pf;
if (_image_content->video->colour_conversion()) {
if (valid_j2k_file (path)) {
AVPixelFormat pf;
if (_image_content->video->colour_conversion()) {
@@
-66,19
+73,21
@@
ImageDecoder::pass ()
/* We can't extract image size from a JPEG2000 codestream without decoding it,
so pass in the image content's size here.
*/
/* 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(), pf)
);
+ _image
= make_shared<J2KImageProxy>(path, _image_content->video->size(), pf
);
} else {
} else {
- _image
.reset (new MagickImageProxy (path)
);
+ _image
= make_shared<FFmpegImageProxy>(path
);
}
}
}
}
- video->emit (_image, _frame_video_position);
+ video->emit (
film(),
_image, _frame_video_position);
++_frame_video_position;
++_frame_video_position;
- return;
+ return
false
;
}
}
+
void
void
-ImageDecoder::seek (ContentTime time, bool)
+ImageDecoder::seek (ContentTime time, bool
accurate
)
{
{
- _frame_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(film()));
}
}