Some include tidying.
[dcpomatic.git] / src / lib / player_video.h
index 0f5e83b10298ff93c85f0b1dee3ee5dc388d282e..935690a08dea475057dd30d647743e6604e6060c 100644 (file)
 
 */
 
-#include <boost/shared_ptr.hpp>
 #include "types.h"
 #include "position.h"
 #include "colour_conversion.h"
 #include "position_image.h"
+extern "C" {
+#include <libavutil/pixfmt.h>
+}
+#include <boost/shared_ptr.hpp>
 
 class Image;
 class ImageProxy;
 class Scaler;
 class Socket;
-class Log;
 class EncodedData;
 
 /** Everything needed to describe a video frame coming out of the player, but with the
@@ -47,14 +49,14 @@ public:
                Scaler const *,
                Eyes,
                Part,
-               ColourConversion
+               boost::optional<ColourConversion>
                );
        
-       PlayerVideo (boost::shared_ptr<cxml::Node>, boost::shared_ptr<Socket>, boost::shared_ptr<Log>);
+       PlayerVideo (boost::shared_ptr<cxml::Node>, boost::shared_ptr<Socket>);
 
        void set_subtitle (PositionImage);
        
-       boost::shared_ptr<Image> image (bool burn_subtitle) const;
+       boost::shared_ptr<Image> image (AVPixelFormat pix_fmt, bool burn_subtitle) const;
 
        void add_metadata (xmlpp::Node* node, bool send_subtitles) const;
        void send_binary (boost::shared_ptr<Socket> socket, bool send_subtitles) const;
@@ -70,7 +72,7 @@ public:
                return _eyes;
        }
 
-       ColourConversion colour_conversion () const {
+       boost::optional<ColourConversion> colour_conversion () const {
                return _colour_conversion;
        }
 
@@ -94,6 +96,6 @@ private:
        Scaler const * _scaler;
        Eyes _eyes;
        Part _part;
-       ColourConversion _colour_conversion;
+       boost::optional<ColourConversion> _colour_conversion;
        PositionImage _subtitle;
 };