std::shared_ptr

[dcpomatic.git] / src / lib / video_content.h

diff --git a/src/lib/video_content.h b/src/lib/video_content.h
index 37223c45788eef070d5ab5780a761e40f763f663..e5bbb5bab066bdcb495700c12aa4faf2cb90151f 100644 (file)
--- a/src/lib/video_content.h
+++ b/src/lib/video_content.h
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2019 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2020 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -22,14 +22,11 @@
 #define DCPOMATIC_VIDEO_CONTENT_H
 
 #include "colour_conversion.h"
-#include "video_content_scale.h"
 #include "dcpomatic_time.h"
 #include "user_property.h"
 #include "types.h"
 #include "content_part.h"
 #include <boost/thread/mutex.hpp>
-#include <boost/weak_ptr.hpp>
-#include <boost/enable_shared_from_this.hpp>
 
 class VideoExaminer;
 class Ratio;
@@ -48,18 +45,20 @@ public:
        static int const FADE_IN;
        static int const FADE_OUT;
        static int const RANGE;
+       static int const CUSTOM_RATIO;
+       static int const CUSTOM_SIZE;
 };
 
-class VideoContent : public ContentPart, public boost::enable_shared_from_this<VideoContent>
+class VideoContent : public ContentPart, public std::enable_shared_from_this<VideoContent>
 {
 public:
        explicit VideoContent (Content* parent);
-       VideoContent (Content* parent, std::vector<boost::shared_ptr<Content> >);
+       VideoContent (Content* parent, std::vector<std::shared_ptr<Content> >);
 
        void as_xml (xmlpp::Node *) const;
        std::string technical_summary () const;
        std::string identifier () const;
-       void take_settings_from (boost::shared_ptr<const VideoContent> c);
+       void take_settings_from (std::shared_ptr<const VideoContent> c);
 
        Frame length () const {
                boost::mutex::scoped_lock lm (_mutex);
@@ -87,7 +86,9 @@ public:
        void set_top_crop (int);
        void set_bottom_crop (int);
 
-       void set_scale (VideoContentScale);
+       void set_custom_ratio (boost::optional<float> ratio);
+       void set_custom_size (boost::optional<dcp::Size> size);
+
        void unset_colour_conversion ();
        void set_colour_conversion (ColourConversion);
 
@@ -127,12 +128,19 @@ public:
                return _crop.bottom;
        }
 
-       /** @return Description of how to scale this content (if indeed it should be scaled) */
-       VideoContentScale scale () const {
+
+       boost::optional<float> custom_ratio () const {
+               boost::mutex::scoped_lock lm (_mutex);
+               return _custom_ratio;
+       }
+
+
+       boost::optional<dcp::Size> custom_size () const {
                boost::mutex::scoped_lock lm (_mutex);
-               return _scale;
+               return _custom_size;
        }
 
+
        boost::optional<ColourConversion> colour_conversion () const {
                boost::mutex::scoped_lock lm (_mutex);
                return _colour_conversion;
@@ -168,25 +176,24 @@ public:
                return _use;
        }
 
+       /* XXX: names for these? */
        dcp::Size size_after_3d_split () const;
        dcp::Size size_after_crop () const;
+       dcp::Size scaled_size (dcp::Size container_size);
 
-       boost::optional<double> fade (boost::shared_ptr<const Film> film, Frame) const;
-
-       void scale_and_crop_to_fit_width (boost::shared_ptr<const Film> film);
-       void scale_and_crop_to_fit_height (boost::shared_ptr<const Film> film);
+       boost::optional<double> fade (std::shared_ptr<const Film> film, Frame) const;
 
-       std::string processing_description (boost::shared_ptr<const Film> film) const;
+       std::string processing_description (std::shared_ptr<const Film> film);
 
        void set_length (Frame);
 
-       void take_from_examiner (boost::shared_ptr<VideoExaminer>);
+       void take_from_examiner (std::shared_ptr<VideoExaminer>);
        void add_properties (std::list<UserProperty> &) const;
 
-       void modify_position (boost::shared_ptr<const Film> film, dcpomatic::DCPTime& pos) const;
+       void modify_position (std::shared_ptr<const Film> film, dcpomatic::DCPTime& pos) const;
        void modify_trim_start (dcpomatic::ContentTime& pos) const;
 
-       static boost::shared_ptr<VideoContent> from_xml (Content* parent, cxml::ConstNodePtr, int);
+       static std::shared_ptr<VideoContent> from_xml (Content* parent, cxml::ConstNodePtr, int);
 
 private:
 
@@ -194,6 +201,9 @@ private:
        friend struct best_dcp_frame_rate_test_single;
        friend struct best_dcp_frame_rate_test_double;
        friend struct audio_sampling_rate_test;
+       friend struct scaled_size_test1;
+       friend struct scaled_size_test2;
+       friend struct scaled_size_legacy_test;
 
        VideoContent (Content* parent, cxml::ConstNodePtr, int);
        void setup_default_colour_conversion ();
@@ -204,7 +214,15 @@ private:
        dcp::Size _size;
        VideoFrameType _frame_type;
        Crop _crop;
-       VideoContentScale _scale;
+       /** ratio to scale cropped image to (or none to guess); i.e. if set, scale to _custom_ratio:1 */
+       boost::optional<float> _custom_ratio;
+       /** size to scale cropped image to; only used if _custom_ratio is none */
+       boost::optional<dcp::Size> _custom_size;
+       /** ratio obtained from an older metadata file; will be used to set up
+        *  _custom_{ratio,size} (or not, if not required) on the first call to
+        *  scaled_size()
+        */
+       boost::optional<float> _legacy_ratio;
        /** Sample aspect ratio obtained from the content file's header, if there is one */
        boost::optional<double> _sample_aspect_ratio;
        bool _yuv;