#467 [framerate_producer] Fixed nb_frames() and frame_number() calculation

[casparcg] / core / producer / transition / transition_producer.cpp

diff --git a/core/producer/transition/transition_producer.cpp b/core/producer/transition/transition_producer.cpp
index c2f73306d02a181c46726c29f66a19b5156541d2..590aca44d248fc02d3c27c9e4179ffb1ca86512c 100644 (file)
--- a/core/producer/transition/transition_producer.cpp
+++ b/core/producer/transition/transition_producer.cpp
@@ -19,7 +19,7 @@
 * Author: Robert Nagy, ronag89@gmail.com
 */
 
-#include "../../stdafx.h"
+#include "../../StdAfx.h"
 
 #include "transition_producer.h"
 
@@ -30,29 +30,28 @@
 
 #include <tbb/parallel_invoke.h>
 
+#include <future>
+
 namespace caspar { namespace core {    
 
 class transition_producer : public frame_producer_base
 {      
        spl::shared_ptr<monitor::subject>       monitor_subject_;
        const field_mode                                        mode_;
-       int                                                                     current_frame_;
+       int                                                                     current_frame_          = 0;
        
        const transition_info                           info_;
                
        spl::shared_ptr<frame_producer>         dest_producer_;
-       spl::shared_ptr<frame_producer>         source_producer_;
+       spl::shared_ptr<frame_producer>         source_producer_        = frame_producer::empty();
 
-       bool                                                            paused_;
+       bool                                                            paused_                         = false;
                
 public:
        explicit transition_producer(const field_mode& mode, const spl::shared_ptr<frame_producer>& dest, const transition_info& info) 
                : mode_(mode)
-               , current_frame_(0)
                , info_(info)
                , dest_producer_(dest)
-               , source_producer_(frame_producer::empty())
-               , paused_(false)
        {
                dest->monitor_output().attach_parent(monitor_subject_);
 
@@ -96,7 +95,7 @@ public:
                *monitor_subject_       << monitor::message("/transition/frame") % current_frame_ % info_.duration
                                                        << monitor::message("/transition/type") % [&]() -> std::string
                                                                                                                                {
-                                                                                                                                       switch(info_.type.value())
+                                                                                                                                       switch(info_.type)
                                                                                                                                        {
                                                                                                                                        case transition_type::mix:              return "mix";
                                                                                                                                        case transition_type::wipe:             return "wipe";
@@ -128,6 +127,11 @@ public:
                return dest_producer_->nb_frames();
        }
 
+       uint32_t frame_number() const override
+       {
+               return dest_producer_->frame_number();
+       }
+
        std::wstring print() const override
        {
                return L"transition[" + source_producer_->print() + L"=>" + dest_producer_->print() + L"]";
@@ -143,7 +147,7 @@ public:
                return dest_producer_->info();
        }
        
-       boost::unique_future<std::wstring> call(const std::vector<std::wstring>& params) override
+       std::future<std::wstring> call(const std::vector<std::wstring>& params) override
        {
                return dest_producer_->call(params);
        }